zabetak commented on a change in pull request #2955:
URL: https://github.com/apache/hive/pull/2955#discussion_r788711952
##########
File path: common/src/java/org/apache/hadoop/hive/conf/HiveConf.java
##########
@@ -1883,6 +1883,8 @@ private static void populateLlapDaemonVarsSet(Set<String>
llapDaemonVarsSetLocal
+ " expressed
as multiple of Local FS write cost"),
HIVE_CBO_COST_MODEL_HDFS_READ("hive.cbo.costmodel.hdfs.read", "1.5",
"Default cost of reading a byte from HDFS;"
+ " expressed
as multiple of Local FS read cost"),
+ HIVE_CBO_RULE_EXCLUSION_REGEX("hive.cbo.rule.exclusion.regex", "",
Review comment:
I would add a bit more information to indicate the purpose of the
property to avoid people playing with it to make the query perform better or
other kind of use-cases. Let's also mention that it is experimental and may be
removed without notice to avoid having to support this kind of "hack" forever.
--
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]
