[CELEBORN-914] Support memory file storage
What changes were proposed in this pull request?
To support memory file storage.
Why are the changes needed?
To improve shuffle performance for small shuffle files.
Design doc: https://docs.google.com/document/d/1SM-oOM0JHEIoRHTYhE9PYH60_1D3NMxDR50LZIM7uW0/edit?usp=sharing
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Pass GA and manually test on a cluster. Here is a brief performance test result. TPC-DS 10G on a 4 node Celeborn cluster(20vCPUs 88 GBs 8xHDD): Without memory file storage: 983s With memory file storage: 791s About 15% performance improvement.
Codecov Report
Attention: Patch coverage is 73.15436% with 40 lines in your changes are missing coverage. Please review.
Project coverage is 49.33%. Comparing base (
21d5698) to head (52cd7d8). Report is 8 commits behind head on main.
Additional details and impacted files
@@ Coverage Diff @@
## main #2300 +/- ##
==========================================
+ Coverage 48.96% 49.33% +0.38%
==========================================
Files 209 211 +2
Lines 13102 13238 +136
Branches 1134 1149 +15
==========================================
+ Hits 6414 6530 +116
- Misses 6270 6276 +6
- Partials 418 432 +14
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Codecov Report
Attention: Patch coverage is 9.49367% with 143 lines in your changes are missing coverage. Please review.
Project coverage is 38.98%. Comparing base (
121395f) to head (53555a1). Report is 51 commits behind head on main.
:exclamation: Current head 53555a1 differs from pull request most recent head 137ab52
Please upload reports for the commit 137ab52 to get more accurate results.
Additional details and impacted files
@@ Coverage Diff @@
## main #2300 +/- ##
==========================================
- Coverage 40.17% 38.98% -1.19%
==========================================
Files 218 219 +1
Lines 13742 13547 -195
Branches 1214 1191 -23
==========================================
- Hits 5520 5280 -240
- Misses 7905 7966 +61
+ Partials 317 301 -16
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@FMX metrics added by this PR is not added to the Celeborn website Monitoring page. Also, should we start adding such changes in the release notes as well. WDYT?