collado-mike commented on PR #741: URL: https://github.com/apache/polaris/pull/741#issuecomment-2594302637
> Please don’t merge a PR if there are open comments. It’s important to work through all feedback, especially for something as impactful as a breaking change. Plus this isn’t an urgent fix, and just filed 2 days ago, we should take the time to think it through carefully before merging. > +1 There are a _lot_ of changes to interfaces and user-facing APIs being pushed and some of them are not really more than cosmetic. I'm all for refactoring and nailing down the right APIs, but these changes should have appropriate review and feedback. If there are open comments, people shouldn't be merging without closing the conversation first. -- 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
