jasonf20 commented on PR #13987: URL: https://github.com/apache/iceberg/pull/13987#issuecomment-3273965153
> This is a pretty Major breakage to the existing API so we would probably have to go through a whole design process here if we did want to change it. I agree with some of the commenters above as well that currently the ability to not need to refresh is encoded int he TableOperations itself which is allowed to skip a refresh really whenever it likes. Hi @RussellSpitzer It's very not helpful that the TableOperations may skip refresh calls if the current implementation doesn't do that, and you will forever be at the mercy of the implementation. Regarding the API change I can easily keep the API unchanged by adding a new method `commitWithResult` that does this and the current `commit` will call that method and not return the result. What do you think about that solution? -- 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
