asolimando commented on code in PR #2966:
URL: https://github.com/apache/hive/pull/2966#discussion_r857601445
##########
common/src/java/org/apache/hadoop/hive/conf/HiveConf.java:
##########
@@ -2547,6 +2547,9 @@ public static enum ConfVars {
"If this config is true only pushed down filters remain in the
operator tree, \n" +
"and the original filter is removed. If this config is false, the
original filter \n" +
"is also left in the operator tree at the original place."),
+
HIVE_JOIN_PUSH_TRANSITIVE_PREDICATES_CONSERVATIVE("hive.optimize.join.transitive.predicates.conservative",
+ false, "Whether to avoid pushing predicates that are hard to simplify.
\n"
Review Comment:
Since we are focusing on the disjunctive predicates (as per another
comment), let's opt for
`hive.optimize.join.disjunctive.transitive.predicates.pushdown`.
As discussed offline, let's leave the property disabled by default, because
it's hard to judge how much benefit/harm there will be for Hive in the wild, we
can revisit the default value later on if we have more evidence that this will
help.
--
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]
