danielcweeks commented on PR #12707: URL: https://github.com/apache/iceberg/pull/12707#issuecomment-2776389996
@manuzhang looking at the test, I think part of it is how the test is designed. There are 5 threads committing, but each thread waits until all other threads have committed before moving on to the next commit. Given that and the wait times, it's very likely they'll exceed the overall time. I'm not sure this is the best way to demonstrate concurrency and contention across commits. It's also a pretty long test (usually takes around 60s on my laptop to complete). Increasing the awaitility time to 30 seconds worked for me (10 seconds was very hit/miss). -- 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
