[Flaky Test][TeZ] WriteBufferManagerTest.testWriteNormal

apache / incubator-uniffle

Uniffle is a high performance, general purpose Remote Shuffle Service.

https://uniffle.apache.org/

Apache License 2.0

370 stars 143 forks source link

[Flaky Test][TeZ] WriteBufferManagerTest.testWriteNormal #1285

Open zuston opened 10 months ago

zuston commented 10 months ago

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Search before asking

[X] I have searched in the issues and found no similar issues.

Describe the flaky test

WriteBufferManagerTest.testWriteNormal

Actions URL

https://github.com/apache/incubator-uniffle/actions/runs/6704550390/job/18217184953

Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

zuston commented 10 months ago

Could you help check this? @zhengchenyu

zhengchenyu commented 10 months ago

Maybe inSendListBytes is updated in another async thread. But I can not catch the second test section in testWriteNormal. why we assert before waitSendFinished? I think before waitSendFinished, waitSendBuffers is non-deterministic. @bin41215 Can you please review it? I am no sure the initial thought.