[CELEBORN-1233] Add unit test to verify data correctness
What changes were proposed in this pull request?
Add a unit test for https://github.com/apache/celeborn/pull/2235
Why are the changes needed?
ditto
Does this PR introduce any user-facing change?
No
How was this patch tested?
org.apache.celeborn.tests.spark.CelebornWorkerRestartSuite
ping @waitinfuture @RexXiong Please take a look. Thanks.
ping @waitinfuture @RexXiong Please take a look. Thanks.
see comments at https://github.com/apache/celeborn/pull/2609
This PR is stale because it has been open 20 days with no activity. Remove stale label or comment or this will be closed in 10 days.
ping @waitinfuture @RexXiong Please take a look. Thanks.
see comments at #2609
Hi, we found that without CELEBORN-1233's PR: #2235 , if a file is deleted, the worker won't throw an exception and this could lead to data correctness issues. There is no unit test for CELEBORN-1233, so I have written one. However, it is not a unit test for #2609.
This PR is stale because it has been open 20 days with no activity. Remove stale label or comment or this will be closed in 10 days.
This issue was closed because it has been staled for 10 days with no activity.