dimas-b commented on code in PR #1724: URL: https://github.com/apache/polaris/pull/1724#discussion_r2116889254
########## persistence/eclipselink/src/main/java/org/apache/polaris/extension/persistence/impl/eclipselink/models/ModelSequenceId.java: ########## Review Comment: Fair enough... although, I guess the deeper implication is that we probably need to documents at some point what modules/classes downstream projects should expect to be "stable" what what is "subject to change without notice". Without defining that across the whole Polaris codebase if would be pretty much impossible to have a common understanding among committers what to flag as compatibility issues in reviews and what is ok :thinking: -- 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
