This is an automated email from the ASF dual-hosted git repository.
beliefer 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 b3a51dfb01 [GLUTEN-10660][VL][FOLLOWUP] Simplify getNativeSessionConf
for GlutenConfig (#11268)
b3a51dfb01 is described below
commit b3a51dfb014b7af47000ec41b5b73444da46feff
Author: Jiaan Geng <[email protected]>
AuthorDate: Tue Dec 9 18:21:40 2025 +0800
[GLUTEN-10660][VL][FOLLOWUP] Simplify getNativeSessionConf for GlutenConfig
(#11268)
---
.../main/scala/org/apache/gluten/config/GlutenConfig.scala | 12 +++++-------
1 file changed, 5 insertions(+), 7 deletions(-)
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 35584ebd0b..40d668091b 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
@@ -551,17 +551,15 @@ 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 confPrefixSession = prefixSessionOf(backendName)
val confPrefix = prefixOf(backendName)
conf
.filter {
case (k, _) =>
- k.startsWith(confPrefix) && !SQLConf.isStaticConfigKey(k)
+ // Backend's dynamic session conf only.
+ k.startsWith(confPrefix) && !SQLConf.isStaticConfigKey(k) ||
+ // put in all gluten velox configs
+ k.startsWith(confPrefixSession)
}
.foreach { case (k, v) => nativeConfMap.put(k, v) }
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
