Skip to content

Commit

Permalink
[VL] Update supportColumnarShuffleExec for Velox to consider enableCo…
Browse files Browse the repository at this point in the history
…lumnarShuffle config (#6055)
  • Loading branch information
acvictor authored Jun 12, 2024
1 parent 1c505df commit c00ffe7
Showing 1 changed file with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit c00ffe7

Please sign in to comment.