viirya commented on code in PR #850:
URL: https://github.com/apache/datafusion-comet/pull/850#discussion_r1722516337
##########
common/src/main/scala/org/apache/comet/CometConf.scala:
##########
@@ -161,6 +161,45 @@ object CometConf extends ShimCometConf {
defaultValue = false,
notes = Some("stddev is slower than Spark's implementation"))
+ val COMET_EXEC_PROJECT_DISABLED: ConfigEntry[Boolean] =
+ createExecDisabledConfig(OPERATOR_PROJECT, defaultValue = false)
+ val COMET_EXEC_FILTER_DISABLED: ConfigEntry[Boolean] =
+ createExecDisabledConfig(OPERATOR_FILTER, defaultValue = false)
+ val COMET_EXEC_SORT_DISABLED: ConfigEntry[Boolean] =
+ createExecDisabledConfig(OPERATOR_SORT, defaultValue = false)
+ val COMET_EXEC_LOCAL_LIMIT_DISABLED: ConfigEntry[Boolean] =
+ createExecDisabledConfig(OPERATOR_LOCAL_LIMIT, defaultValue = false)
+ val COMET_EXEC_GLOBAL_LIMIT_DISABLED: ConfigEntry[Boolean] =
+ createExecDisabledConfig(OPERATOR_GLOBAL_LIMIT, defaultValue = false)
+ val COMET_EXEC_BROADCAST_HASH_JOIN_DISABLED: ConfigEntry[Boolean] =
+ createExecDisabledConfig(OPERATOR_BROADCAST_HASH_JOIN, defaultValue =
false)
+ val COMET_EXEC_BROADCAST_EXCHANGE_DISABLED: ConfigEntry[Boolean] =
+ createExecDisabledConfig(OPERATOR_BROADCAST_EXCHANGE, defaultValue = false)
+ val COMET_EXEC_HASH_JOIN_DISABLED: ConfigEntry[Boolean] =
+ createExecDisabledConfig(OPERATOR_HASH_JOIN, defaultValue = false)
+ val COMET_EXEC_SORT_MERGE_JOIN_DISABLED: ConfigEntry[Boolean] =
Review Comment:
In `isCometOperatorEnabled`, currently `disabled` configs have higher
priority.
But I agree that it would make confusion.
Another proposal would be to disable the operator if `enabled` config is
false even `COMET_EXEC_ALL_OPERATOR_ENABLED` is true.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
