chinnaraolalam commented on code in PR #4881:
URL: https://github.com/apache/hive/pull/4881#discussion_r1411550394
##########
common/src/java/org/apache/hadoop/hive/conf/HiveConf.java:
##########
@@ -1299,9 +1299,10 @@ public static enum ConfVars {
*/
@Deprecated
METASTORE_BATCH_RETRIEVE_MAX("hive.metastore.batch.retrieve.max", 300,
+ new RangeValidator(1, null),
"Maximum number of objects (tables/partitions) can be retrieved from
metastore in one batch. \n" +
"The higher the number, the less the number of round trips is needed
to the Hive metastore server, \n" +
- "but it may also cause higher memory requirement at the client side."),
+ "but it may also cause higher memory requirement at the client side.
Batch value should be > 0 "),
Review Comment:
Where did we use RangeValidator and upper value can be null?
--
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]
