kbendick commented on issue #3112: URL: https://github.com/apache/iceberg/issues/3112#issuecomment-919729702
I think placing the +2 is a decent idea, if that works. If we disable the +2 from the checks, will the test then fail on situations where it was currently stuck in an infinite loop? I also noticed that there is a callback for checkpoint aborted events. Could we possibly hook into that and on failure check if the union of checkpoint events and events yet to be checkpoint is complete for only 1 of the 2 topologies? Seems really flakey, but maybe you can think of something better than I can for the checkpoint aborted exceptions. If disabling the +2 wouldn't leave the test flakey, I 100% support that decision. -- 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
