flyrain commented on PR #3256: URL: https://github.com/apache/polaris/pull/3256#issuecomment-3788429475
@pingtimeout Thanks for laying out the proposal, I appreciate the effort to move things forward. I respectfully disagree with merging #3256 as is. While the contribution has merit, there are still unresolved concerns around scope, dependency choices, and validation at scale. These are not just process questions, they materially affect maintainability and whether this is something we would accept from any contributor. I agree that we should have a broader dev list discussion about review practices, but in the meantime I think the safer path is to split the work into smaller, clearly scoped PRs and address the remaining concerns directly before merging. That keeps the bar consistent and avoids relying on follow ups for core design issues. -- 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]
