[ https://issues.apache.org/jira/browse/HBASE-7271?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13512003#comment-13512003 ]
nkeywal commented on HBASE-7271: -------------------------------- bq. nit: why the increase in number of workers ? It's far from nit ;-). I think 5 is not enough. I've done tests bumping the number of threads everywhere in HBASE-7248. I will redo it after once HBASE-7247 & HBASE-7246 are committed, so identify something reasonably optimal, but I wanted to put a first stone there... > Have a single executor for all zkWorkers in the assignment manager > ------------------------------------------------------------------ > > Key: HBASE-7271 > URL: https://issues.apache.org/jira/browse/HBASE-7271 > Project: HBase > Issue Type: Bug > Components: master, Region Assignment > Affects Versions: 0.96.0 > Reporter: nkeywal > Assignee: nkeywal > Fix For: 0.96.0 > > Attachments: 7271.v1.patch, 7271.v2.patch, 7271.v2.patch > > > The current strategy is to have an array of monothreaded executor, and hash > the zk path to ensure that there are no two events on the same region > executed in parallel > I think a single executor, as presented in the attached patch, is better > because: > - we're guaranteed to use all threads at any time > - if managing one of the event takes longer that expected, the slowness is > limited to this region, and not to all regions that have the same > hashed/moduloed code > - For the nodeChildrenChanged, there is no need to choose randomly one of the > worker (or, once again, the risk to get stuck if one of the event takes time > to be managed). -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira