New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix alluxio.util.executor.UniqueBlockingQueueTest.concurrentTest fail #18668

Open

gp1314 wants to merge 1 commit into Alluxio:master-2.x from gp1314:dev/gup/fix_unit_test

Contributor

gp1314 commented Aug 2, 2024

What changes are proposed in this pull request?

Fix unit test alluxio.util.executor.UniqueBlockingQueueTest.concurrentTest failure

Why are the changes needed?

Wait for the thread to complete after executing executor.shutdown()
Checking test.size() may not pass if the thread has not completed execution

Does this PR introduce any user facing changes?

no user facing changes


          Fix alluxio.util.executor.UniqueBlockingQueueTest.concurrentTest fail

bef5f0a

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet