[GitHub] [hive] nrg4878 commented on a diff in pull request #4038: Hive 27011 - Default value of PartitionManagementTask frequency should be set to higher value

["nrg4878 (via GitHub)" <gi...@apache.org>]

nrg4878 commented on code in PR #4038:
URL: https://github.com/apache/hive/pull/4038#discussion_r1101678244


##########
standalone-metastore/metastore-common/src/main/java/org/apache/hadoop/hive/metastore/conf/MetastoreConf.java:
##########
@@ -1152,7 +1152,7 @@ public enum ConfVars {
     // Partition management task params
     PARTITION_MANAGEMENT_TASK_FREQUENCY("metastore.partition.management.task.frequency",
       "metastore.partition.management.task.frequency",
-      300, TimeUnit.SECONDS, "Frequency at which timer task runs to do automatic partition management for tables\n" +
+      21600, TimeUnit.SECONDS, "Frequency at which timer task runs to do automatic partition management for tables\n" +

Review Comment:
   @rtrivedi12  we can also specify the time in hours instead of seconds.
   Can we change this to 6, TimeUnit.HOURS instead?



-- 
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: gitbox-unsubscribe@hive.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org