From c00ffe7db44dbcd6f1f0a41067307adeeaccad47 Mon Sep 17 00:00:00 2001 From: Ankita Victor Date: Wed, 12 Jun 2024 14:48:32 +0530 Subject: [PATCH] [VL] Update supportColumnarShuffleExec for Velox to consider enableColumnarShuffle config (#6055) --- .../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 060a0c4cbd34..6bc7df98cca2 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