mayankkunwar commented on code in PR #4758:
URL: https://github.com/apache/hive/pull/4758#discussion_r1345709347
##########
common/src/test/org/apache/hadoop/hive/conf/TestHiveConf.java:
##########
@@ -95,6 +95,9 @@ public void testConfProperties() throws Exception {
// Test HiveConf property variable substitution in hive-site.xml
checkHiveConf("test.var.hiveconf.property",
ConfVars.DEFAULTPARTITIONNAME.getDefaultValue());
+
+ // Test if LLAP_LRFU_HOTBUFFERS_PERCENTAGE is defined in HiveConf
+
Assert.assertTrue(HiveConf.getLlapDaemonConfVars().contains(ConfVars.LLAP_LRFU_HOTBUFFERS_PERCENTAGE.varname));
Review Comment:
Name starts with "LLAP_" would also not work because there are some conf
vars which do not fall into this category.
I see your point but for now I am not sure if we could do anything on this
to make this future proof.
Like for example there is a conf
"LLAP_TASK_SCHEDULER_AM_REGISTRY_KEYTAB_FILE" which is not considered in
llapDaemonVarsSetLocal
--
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]
