rdblue commented on PR #9230: URL: https://github.com/apache/iceberg/pull/9230#issuecomment-1861257079
> I think this fix is still relevant as the test fails without it. To which test case are you referring? I didn't think any of them were valid uses of the API because commit was called multiple times and the behavior is undefined. > The only change that is specifically for this use case is the forceReApply I added to the transaction What is the purpose of `forceReApply`? I think that we should focus on fixing the bug and then discuss cases where `commit` or `commitTransaction` are called multiple times. The important thing is getting a fix we can get in a patch release. -- 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