saintstack commented on a change in pull request #955: HBASE-23597 Give high
priority for meta assign procedure and ServerCr…
URL: https://github.com/apache/hbase/pull/955#discussion_r360475374
##########
File path:
hbase-procedure/src/main/java/org/apache/hadoop/hbase/procedure2/ProcedureScheduler.java
##########
@@ -95,6 +95,20 @@
*/
boolean hasRunnables();
+ /**
+ * Fetch one high priority Procedure from the queue
+ * @return the Procedure to execute, or null if nothing present.
+ */
+ Procedure pollHighPriority();
+
+ /**
+ * Fetch one high priority Procedure from the queue
+ * @param timeout how long to wait before giving up, in units of unit
+ * @param unit a TimeUnit determining how to interpret the timeout parameter
+ * @return the Procedure to execute, or null if nothing present.
+ */
+ Procedure pollHighPriority(long timeout, TimeUnit unit);
Review comment:
What is high priority? We have priority on rpcs with a gradient where 200 is
high priority. Here there only gradient is high or not high? And only the
assign of meta is high priority?
----------------------------------------------------------------
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:
[email protected]
With regards,
Apache Git Services
