danielcweeks commented on PR #9872: URL: https://github.com/apache/iceberg/pull/9872#issuecomment-1986606686
> I am quite confused about this situation. We literally merged a spec change, and then immediately published a revision which completely rewrites the description. We could have discussed this on devlist if there is any additional disagreements instead of just putting up a PR without even informing the original author or requesting original author and reviewers for review. This is even worse given that there is already an attempt for the original author to fix the comments Ryan made after the original PR is merged. This does not sound very community friendly to me. @jackye1995 I wasn't aware of the other PR, they were created relatively close in time. I originally thought it would be a small clarification and it expanded as the comments came in. I don't there's really a disagreement in terms of the behavior, it was more that there were a number of concerns raised that the content wasn't clear. -- 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...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
