[ 
https://issues.apache.org/jira/browse/HBASE-18109?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16037359#comment-16037359
 ] 

Yi Liang commented on HBASE-18109:
----------------------------------

Hi [~stack]
After reviewing the current implementation, I have some thoughts on this jira. 

>From current implementation, there are two kinds of Procedure, one is 
>ServerProcedure, and one is TableProcedure; For ProcedureExecutor, 
>ServerProcedure will always be executed before TableProcedure.  

Right now, there are only one ServerProcedure, which is ServerCrashProcedure
For TableProcedure, it is really hard to see which one has priority, for 
example, We can not decided CreateTableProcedure has higher priority than 
DeletetTableProcedure, it should only depends on which Procedure submit first.

And the priority could be useful for some bulk operations for certain 
procedures,  like bulk region assign, when master start or server crashed, 
hbase can create and sort AssignProcedures based on table priority, and submit 
them to ProcedureExecutor.

For Phil's problem, it is more like priority for Table, not for Procedure. The 
table whose higher priority will be assigned first.  So maybe we need first add 
priority into Table.

[~stack], or do you have any other situation,  the priority can be really 
helpful for procedure. 

My idea is that 
For Procedure, we can still add facility to mark a priority on a Procedure, and 
maybe now it is only used in AssignProcedure. 
And ProcedureExecutor still work as the current way, so the priority only used 
in submitting the sorted bulk AssignProcedureS.  
For Table, add Priority into it, and this property can be only used for bulk 
assign right now. 

> Assign system tables first (priority)
> -------------------------------------
>
>                 Key: HBASE-18109
>                 URL: https://issues.apache.org/jira/browse/HBASE-18109
>             Project: HBase
>          Issue Type: Sub-task
>          Components: Region Assignment
>    Affects Versions: 2.0.0
>            Reporter: stack
>            Assignee: Yi Liang
>            Priority: Critical
>             Fix For: 2.0.0
>
>         Attachments: HBASE-18109-V1.patch
>
>
> Need this for stuff like the RSGroup table, etc. Assign these ahead of 
> user-space regions.
> From 'Handle sys table assignment first (e.g. acl, namespace, rsgroup); 
> currently only hbase:meta is first.' of 
> https://docs.google.com/document/d/1eVKa7FHdeoJ1-9o8yZcOTAQbv0u0bblBlCCzVSIn69g/edit#heading=h.oefcyphs0v0x



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to