apache / celeborn

Apache Celeborn is an elastic and high-performance service for shuffle and spilled data.
https://celeborn.apache.org/
Apache License 2.0
893 stars 361 forks source link

[CELEBORN-1705][0.5][FOLLWUP] Fix cannot find symbol #2919

Closed turboFei closed 1 week ago

turboFei commented 1 week ago

What changes were proposed in this pull request?

Fix branch-0.5 build on commit: https://github.com/apache/celeborn/commit/28b2f56e185882a73012f9c8317b5230a69b7188

Why are the changes needed?

Fix branch-0.5 build.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

GA

turboFei commented 1 week ago

Hi @FMX, I force pushed this fix to branch-0.5, it should be fine?

FMX commented 1 week ago

It's OK but do not do that again. I discovered I solved the cherry-pick conflicts but forgot to compile them before pushing. I'll compile codes before pushing next time.

turboFei commented 1 week ago

ok, thanks for the reminder.