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_r360475951
########## File path: hbase-server/src/main/java/org/apache/hadoop/hbase/master/procedure/MasterProcedureScheduler.java ########## @@ -135,7 +139,11 @@ protected void enqueue(final Procedure proc, final boolean addFront) { doAdd(tableRunQueue, getTableQueue(getTableName(proc)), proc, addFront); } else if (isServerProcedure(proc)) { ServerProcedureInterface spi = (ServerProcedureInterface) proc; - doAdd(serverRunQueue, getServerQueue(spi.getServerName(), spi), proc, addFront); + if (spi.hasMetaTableRegion()) { + doAdd(serverHighPriorityRunQueue, getServerQueue(proc), proc, addFront); Review comment: Does this make it so all procedures associated with this server are '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: us...@infra.apache.org With regards, Apache Git Services