[GitHub] [hbase] virajjasani commented on a change in pull request #2272: HBASE-24898 Can not set 23:00~24:00 as offpeak hour now

[GitBox <gi...@apache.org>]

virajjasani commented on a change in pull request #2272:
URL: https://github.com/apache/hbase/pull/2272#discussion_r472236297



##########
File path: hbase-common/src/main/resources/hbase-default.xml
##########
@@ -917,13 +917,13 @@ possible configurations would overwhelm and obscure the important.
   <property>
     <name>hbase.offpeak.start.hour</name>
     <value>-1</value>
-    <description>The start of off-peak hours, expressed as an integer between 0 and 23, inclusive.
+    <description>The start of off-peak hours, expressed as an integer between 0 and 24, inclusive.
       Set to -1 to disable off-peak.</description>
   </property>
   <property>
     <name>hbase.offpeak.end.hour</name>
     <value>-1</value>
-    <description>The end of off-peak hours, expressed as an integer between 0 and 23, inclusive. Set
+    <description>The end of off-peak hours, expressed as an integer between 0 and 24, exclusive. Set

Review comment:
       How is this `exclusive` for 24 based on `return 0 <= hour && hour <= 24; ` condition?




----------------------------------------------------------------
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.

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