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

philo 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 8c78bb097f [GLUTEN-10660][VL] Fix passing Velox session configurations 
on hash build optimizations (#11134)
8c78bb097f is described below

commit 8c78bb097f6e48a0846f10ffddd848334c52c743
Author: Yuan <[email protected]>
AuthorDate: Sat Dec 6 11:14:54 2025 +0800

    [GLUTEN-10660][VL] Fix passing Velox session configurations on hash build 
optimizations (#11134)
---
 .../src/main/scala/org/apache/gluten/config/VeloxConfig.scala       | 5 +++--
 .../src/main/scala/org/apache/gluten/config/GlutenConfig.scala      | 6 ++++++
 2 files changed, 9 insertions(+), 2 deletions(-)

diff --git 
a/backends-velox/src/main/scala/org/apache/gluten/config/VeloxConfig.scala 
b/backends-velox/src/main/scala/org/apache/gluten/config/VeloxConfig.scala
index 6ede352f53..5444059a0a 100644
--- a/backends-velox/src/main/scala/org/apache/gluten/config/VeloxConfig.scala
+++ b/backends-velox/src/main/scala/org/apache/gluten/config/VeloxConfig.scala
@@ -547,8 +547,9 @@ object VeloxConfig extends ConfigRegistry {
   val VELOX_HASHMAP_ABANDON_BUILD_DUPHASH_MIN_PCT =
     buildConf("spark.gluten.velox.abandonbuild.noduphashminpct")
       .experimental()
-      .doc("Experimental: abandon hashmap build if duplicated rows are more 
than this percentile.")
-      .doubleConf
+      .doc("Experimental: abandon hashmap build if duplicated rows are more 
than this percentile." +
+        "Value is integer based and range is [0, 100].")
+      .intConf
       .createWithDefault(0)
 
   val QUERY_TRACE_ENABLED = 
buildConf("spark.gluten.sql.columnar.backend.velox.queryTraceEnabled")
diff --git 
a/gluten-substrait/src/main/scala/org/apache/gluten/config/GlutenConfig.scala 
b/gluten-substrait/src/main/scala/org/apache/gluten/config/GlutenConfig.scala
index 340c490993..4145b526b9 100644
--- 
a/gluten-substrait/src/main/scala/org/apache/gluten/config/GlutenConfig.scala
+++ 
b/gluten-substrait/src/main/scala/org/apache/gluten/config/GlutenConfig.scala
@@ -456,6 +456,7 @@ object GlutenConfig extends ConfigRegistry {
   }
 
   def prefixOf(backendName: String): String = 
s"spark.gluten.sql.columnar.backend.$backendName"
+  def prefixSessionOf(backendName: String): String = 
s"spark.gluten.$backendName"
 
   private lazy val nativeKeys = Set(
     DEBUG_ENABLED.key,
@@ -557,6 +558,11 @@ object GlutenConfig extends ConfigRegistry {
           nativeConfMap
             .put(SQLConf.LEGACY_TIME_PARSER_POLICY.key, 
v.toUpperCase(Locale.ROOT)))
 
+    // put in all gluten velox configs
+    conf
+      .filter(_._1.startsWith(prefixSessionOf(backendName)))
+      .foreach(entry => nativeConfMap.put(entry._1, entry._2))
+
     // Backend's dynamic session conf only.
     val confPrefix = prefixOf(backendName)
     conf


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to