search

Alluxio / alluxio

Alluxio, data orchestration for analytics and machine learning in the cloud
https://www.alluxio.io
Apache License 2.0
6.87k stars 2.94k forks source link

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

Open gp1314 opened 3 months ago

gp1314 commented 3 months ago

What changes are proposed in this pull request?

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

Why are the changes needed?

  1. Wait for the thread to complete after executing executor.shutdown()
  2. 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