InvisibleProgrammer commented on code in PR #5370:
URL: https://github.com/apache/hive/pull/5370#discussion_r1693204413
##########
common/src/java/org/apache/hadoop/hive/conf/HiveConf.java:
##########
@@ -3865,6 +3865,8 @@ public static enum ConfVars {
"get old behavior, if desired. See, test-case in patch for
HIVE-6689."),
HIVE_LINEAGE_INFO("hive.lineage.hook.info.enabled", false,
"Whether Hive provides lineage information to hooks."),
+ HIVE_LINEAGE_STATEMENT_FILTER("hive.lineage.statement.filter", "All",
Review Comment:
I don't know if we have a rule about it or not. But can the default value be
lowercase?
I would love to avoid future confusion about if is that All, all or ALL.
Also, could you please put some description and and example values there
that can help us to understand what that config value exactly does?
--
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]
