viirya commented on code in PR #36102:
URL: https://github.com/apache/spark/pull/36102#discussion_r846349679
##########
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala:
##########
@@ -364,7 +364,7 @@ object SQLConf {
"to insert a bloom filter in the other side to reduce the amount of
shuffle data.")
.version("3.3.0")
.booleanConf
- .createWithDefault(false)
+ .createWithDefault(true)
Review Comment:
Hmm, is this change for master or 3.3? Seems this is a new feature added in
3.3, isn't it safer to keep it as false by default? Usually we choose to be
conservative instead of aggressive on enabling new feature.
--
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]
