[CELEBORN-1823] Remove unused remote-shuffle.job.min.memory-per-parti…

…tion and remote-shuffle.job.min.memory-per-gate ### What changes were proposed in this pull request? Remove unused plugin config `remote-shuffle.job.min.memory-per-partition` and `remote-shuffle.job.min.memory-per-gate`. ### Why are the changes needed? `remote-shuffle.job.min.memory-per-partition` and `remote-shuffle.job.min.memory-per-gate` are not alternative for `CelebornConf`. Therefore, `remote-shuffle.job.min.memory-per-partition` and `remote-shuffle.job.min.memory-per-gate` could be removed. ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? CI. Closes #3053 from SteNicholas/CELEBORN-1823. Authored-by: SteNicholas <[email protected]> Signed-off-by: Shuang <[email protected]>
apache · Jan 7, 2025 · 2962c11 · 2962c11
1 parent 19fecad
commit 2962c11
Showing 1 changed file with 0 additions and 2 deletions.
diff --git a/client-flink/common/src/main/java/org/apache/celeborn/plugin/flink/utils/FlinkUtils.java b/client-flink/common/src/main/java/org/apache/celeborn/plugin/flink/utils/FlinkUtils.java
@@ -32,8 +32,6 @@ public class FlinkUtils {
   private static final JobID ZERO_JOB_ID = new JobID(0, 0);
   public static final Set<String> pluginConfNames =
       ImmutableSet.of(
-          "remote-shuffle.job.min.memory-per-partition",
-          "remote-shuffle.job.min.memory-per-gate",
           "remote-shuffle.job.concurrent-readings-per-gate",
           "remote-shuffle.job.memory-per-partition",
           "remote-shuffle.job.memory-per-gate",