scarlin-cloudera commented on code in PR #4101:
URL: https://github.com/apache/hive/pull/4101#discussion_r1131261309
##########
common/src/java/org/apache/hadoop/hive/conf/HiveConf.java:
##########
@@ -2366,7 +2366,7 @@ public static enum ConfVars {
"Whether Hive enables the optimization about converting common join
into mapjoin based on the input file size. \n" +
"If this parameter is on, and the sum of size for n-1 of the
tables/partitions for a n-way join is smaller than the\n" +
"specified size, the join is directly converted to a mapjoin (there is
no conditional task)."),
- HIVE_CONVERT_ANTI_JOIN("hive.auto.convert.anti.join", true,
+ HIVE_CONVERT_ANTI_JOIN("hive.auto.convert.anti.join", false,
"Whether Hive enables the optimization about converting join with null
filter to anti join"),
Review Comment:
So regarding the first point. I'm kinda feeling like the tests should match
the default behavior. I wrote "temporarily" in the description, but not sure
when that will happen. I think the tests should match the default behavior.
Regarding the second point, yeah, that's a good idea and I'll add that.
--
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]
