dimas-b commented on code in PR #2749:
URL: https://github.com/apache/polaris/pull/2749#discussion_r2402895845


##########
CHANGELOG.md:
##########
@@ -53,6 +51,11 @@ request adding CHANGELOG notes for breaking (!) changes and 
possibly other secti
 ### Deprecations
 
 * The property `polaris.active-roles-provider.type` is deprecated and has no 
effect anymore.
+- The EclipseLink Persistence implementation has been deprecated since 1.0.0 
and will be completely removed
+  in 1.3.0 or in 2.0.0 (whichever happens earlier).
+- The legacy management endpoints at `/metrics` and `/healthcheck` have been 
deprecated in 1.2.0 and will be
+  completely removed in 1.3.0 or in 2.0.0 (whichever happens earlier). Please 
use the standard management

Review Comment:
   As for the number of releases to "wait", I'm not sure it helps users so 
much. People who upgrade often will be able to deal with changes. People who do 
not upgrade to avoid changes, will be suffering from lack of evolution / bug 
fixes in EL, so it's not a sound strategy IMHO.
   
   As for the endpoints, as I commented in some other thread, it's really a 
backend change. It only affects the "devops" personas and can be applied 
holistically without any visible client-side effects... so again I do not see a 
strong reason to defer this change.
   
   ... happy to discuss on `dev` ML too :slightly_smiling_face: 



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to