This is an automated email from the ASF dual-hosted git repository.

klion26 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/amoro.git


The following commit(s) were added to refs/heads/master by this push:
     new 4dda58a5e Fixed the memory display error in Spark optimizer resource 
allocation.
4dda58a5e is described below

commit 4dda58a5efae7728a78ce5e2c509c74ff05fcce5
Author: Darcy <[email protected]>
AuthorDate: Mon Jul 21 16:05:18 2025 +0800

    Fixed the memory display error in Spark optimizer resource allocation.
---
 .../main/java/org/apache/amoro/optimizer/spark/SparkOptimizer.java   | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git 
a/amoro-optimizer/amoro-optimizer-spark/src/main/java/org/apache/amoro/optimizer/spark/SparkOptimizer.java
 
b/amoro-optimizer/amoro-optimizer-spark/src/main/java/org/apache/amoro/optimizer/spark/SparkOptimizer.java
index d8c8c91a1..2eb38c028 100644
--- 
a/amoro-optimizer/amoro-optimizer-spark/src/main/java/org/apache/amoro/optimizer/spark/SparkOptimizer.java
+++ 
b/amoro-optimizer/amoro-optimizer-spark/src/main/java/org/apache/amoro/optimizer/spark/SparkOptimizer.java
@@ -56,7 +56,10 @@ public class SparkOptimizer extends Optimizer {
     // calculate optimizer memory allocation
     int driverMemory = 
Utils.memoryStringToMb(jsc.getConf().get("spark.driver.memory", "1g"));
     int executorMemory = 
Utils.memoryStringToMb(jsc.getConf().get("spark.executor.memory", "1g"));
-    config.setMemorySize(driverMemory + config.getExecutionParallel() * 
executorMemory);
+    int executorCores = jsc.getConf().getInt("spark.executor.cores", 1);
+    int executionParallel = config.getExecutionParallel();
+    int executorNum = (int) Math.ceil((double) executionParallel / 
executorCores);
+    config.setMemorySize(driverMemory + executorNum * executorMemory);
 
     SparkOptimizer optimizer = new SparkOptimizer(config, jsc);
     OptimizerToucher toucher = optimizer.getToucher();

Reply via email to