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

felixybw pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/incubator-gluten.git


The following commit(s) were added to refs/heads/main by this push:
     new 106a57a6fd [GLUTEN-10364][VL] Fix get unset GlutenCoreConfig (#10365)
106a57a6fd is described below

commit 106a57a6fdfb1714d80d16057e171b9e0d280b0a
Author: Mingliang Zhu <zhuml1...@gmail.com>
AuthorDate: Wed Aug 6 03:35:02 2025 +0800

    [GLUTEN-10364][VL] Fix get unset GlutenCoreConfig (#10365)
    
    fix bug and simplify the usage of GlutenCoreConfig in Java.
---
 .../memory/memtarget/ThrowOnOomMemoryTarget.java   | 28 +++++++++-------------
 1 file changed, 11 insertions(+), 17 deletions(-)

diff --git 
a/gluten-core/src/main/java/org/apache/gluten/memory/memtarget/ThrowOnOomMemoryTarget.java
 
b/gluten-core/src/main/java/org/apache/gluten/memory/memtarget/ThrowOnOomMemoryTarget.java
index b03e23a898..8ebedd5418 100644
--- 
a/gluten-core/src/main/java/org/apache/gluten/memory/memtarget/ThrowOnOomMemoryTarget.java
+++ 
b/gluten-core/src/main/java/org/apache/gluten/memory/memtarget/ThrowOnOomMemoryTarget.java
@@ -16,7 +16,7 @@
  */
 package org.apache.gluten.memory.memtarget;
 
-import org.apache.gluten.config.GlutenCoreConfig$;
+import org.apache.gluten.config.GlutenCoreConfig;
 
 import org.apache.spark.memory.SparkMemoryUtil;
 import org.apache.spark.sql.internal.SQLConf;
@@ -62,47 +62,41 @@ public class ThrowOnOomMemoryTarget implements MemoryTarget 
{
         .append(
             String.format(
                 "\t%s=%s",
-                
GlutenCoreConfig$.MODULE$.COLUMNAR_OFFHEAP_SIZE_IN_BYTES().key(),
+                GlutenCoreConfig.COLUMNAR_OFFHEAP_SIZE_IN_BYTES().key(),
                 reformatBytes(
                     SQLConf.get()
-                        .getConfString(
-                            
GlutenCoreConfig$.MODULE$.COLUMNAR_OFFHEAP_SIZE_IN_BYTES().key()))))
+                        
.getConfString(GlutenCoreConfig.COLUMNAR_OFFHEAP_SIZE_IN_BYTES().key()))))
         .append(System.lineSeparator())
         .append(
             String.format(
                 "\t%s=%s",
-                
GlutenCoreConfig$.MODULE$.COLUMNAR_TASK_OFFHEAP_SIZE_IN_BYTES().key(),
+                GlutenCoreConfig.COLUMNAR_TASK_OFFHEAP_SIZE_IN_BYTES().key(),
                 reformatBytes(
                     SQLConf.get()
                         .getConfString(
-                            GlutenCoreConfig$.MODULE$
-                                .COLUMNAR_TASK_OFFHEAP_SIZE_IN_BYTES()
-                                .key()))))
+                            
GlutenCoreConfig.COLUMNAR_TASK_OFFHEAP_SIZE_IN_BYTES().key()))))
         .append(System.lineSeparator())
         .append(
             String.format(
                 "\t%s=%s",
-                
GlutenCoreConfig$.MODULE$.COLUMNAR_CONSERVATIVE_TASK_OFFHEAP_SIZE_IN_BYTES().key(),
+                
GlutenCoreConfig.COLUMNAR_CONSERVATIVE_TASK_OFFHEAP_SIZE_IN_BYTES().key(),
                 reformatBytes(
                     SQLConf.get()
                         .getConfString(
-                            GlutenCoreConfig$.MODULE$
-                                
.COLUMNAR_CONSERVATIVE_TASK_OFFHEAP_SIZE_IN_BYTES()
+                            
GlutenCoreConfig.COLUMNAR_CONSERVATIVE_TASK_OFFHEAP_SIZE_IN_BYTES()
                                 .key()))))
         .append(System.lineSeparator())
         .append(
             String.format(
                 "\t%s=%s",
-                GlutenCoreConfig$.MODULE$.SPARK_OFFHEAP_ENABLED_KEY(),
-                
SQLConf.get().getConfString(GlutenCoreConfig$.MODULE$.SPARK_OFFHEAP_ENABLED_KEY())))
+                GlutenCoreConfig.SPARK_OFFHEAP_ENABLED_KEY(),
+                
SQLConf.get().getConfString(GlutenCoreConfig.SPARK_OFFHEAP_ENABLED_KEY())))
         .append(System.lineSeparator())
         .append(
             String.format(
                 "\t%s=%s",
-                
GlutenCoreConfig$.MODULE$.DYNAMIC_OFFHEAP_SIZING_ENABLED().key(),
-                SQLConf.get()
-                    .getConfString(
-                        
GlutenCoreConfig$.MODULE$.DYNAMIC_OFFHEAP_SIZING_ENABLED().key())))
+                GlutenCoreConfig.DYNAMIC_OFFHEAP_SIZING_ENABLED().key(),
+                GlutenCoreConfig.get().dynamicOffHeapSizingEnabled()))
         .append(System.lineSeparator());
     // Dump all consumer usages to exception body
     errorBuilder.append(SparkMemoryUtil.dumpMemoryTargetStats(target));


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@gluten.apache.org
For additional commands, e-mail: commits-h...@gluten.apache.org

Reply via email to