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

marong 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 c00ffe7db [VL] Update supportColumnarShuffleExec for Velox to consider 
enableColumnarShuffle config (#6055)
c00ffe7db is described below

commit c00ffe7db44dbcd6f1f0a41067307adeeaccad47
Author: Ankita Victor <[email protected]>
AuthorDate: Wed Jun 12 14:48:32 2024 +0530

    [VL] Update supportColumnarShuffleExec for Velox to consider 
enableColumnarShuffle config (#6055)
---
 .../scala/org/apache/gluten/backendsapi/velox/VeloxBackend.scala    | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git 
a/backends-velox/src/main/scala/org/apache/gluten/backendsapi/velox/VeloxBackend.scala
 
b/backends-velox/src/main/scala/org/apache/gluten/backendsapi/velox/VeloxBackend.scala
index 060a0c4cb..6bc7df98c 100644
--- 
a/backends-velox/src/main/scala/org/apache/gluten/backendsapi/velox/VeloxBackend.scala
+++ 
b/backends-velox/src/main/scala/org/apache/gluten/backendsapi/velox/VeloxBackend.scala
@@ -351,9 +351,9 @@ object VeloxBackendSettings extends BackendSettingsApi {
   }
 
   override def supportColumnarShuffleExec(): Boolean = {
-    GlutenConfig.getConf.isUseColumnarShuffleManager ||
-    GlutenConfig.getConf.isUseCelebornShuffleManager ||
-    GlutenConfig.getConf.isUseUniffleShuffleManager
+    GlutenConfig.getConf.enableColumnarShuffle && 
(GlutenConfig.getConf.isUseColumnarShuffleManager
+      || GlutenConfig.getConf.isUseCelebornShuffleManager
+      || GlutenConfig.getConf.isUseUniffleShuffleManager)
   }
 
   override def enableJoinKeysRewrite(): Boolean = false


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

Reply via email to