adutra commented on code in PR #1724: URL: https://github.com/apache/polaris/pull/1724#discussion_r2121276660
########## persistence/eclipselink/src/main/java/org/apache/polaris/extension/persistence/impl/eclipselink/models/ModelSequenceId.java: ########## Review Comment: I think we still need at least some degree of guarantee for Java APIs as well. There are a few vendors integrating Polaris in their own offerings, and a change in public method in any service module (not only core) may create problems downstream. Configuration changes are another area where some guarantees should exist. That said, I also think that until we get to 1.0.0, any changes should be allowed as long as they are clearly communicated. So for now I'm OK with the proposed refactoring here. -- 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: issues-unsubscr...@polaris.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
