in 98, many operations are executed concurrently using zkWorkers, so some pools are not used. can be removed to make code clean and easy to read
thanks On 2016-06-03 21:17 , Ted Yu Wrote: I checked head of 0.98 branch. For the two 'new OpenedRegionHandler()' calls in AssignmentManager.java, process() is invoked directly. Looks like you're right. On Thu, Jun 2, 2016 at 10:58 PM, WangYQ <wangyongqiang0...@163.com> wrote: > RS_ZK_REGION_OPENED (4, ExecutorType.MASTER_OPEN_REGION),there is a > relation between the enevt type "RS_ZK_REGION_OPENED" and pool > "MASTER_OPEN_REGION", but in hbase 0.98.10, we do not use these pools > anymoreexamples are :class AssignmentManagermethod: handleRegionin case > RS_ZK_REGION_OPENED, we construct a OpendRegionHandler and call the process > method, not submit this handler to pool(in class AssignmentManager, line > 1078) > > > > > > > > > At 2016-06-02 21:29:09, "Ted Yu" <yuzhih...@gmail.com> wrote: > >Have you seen this line in EventType.java ? > > > > RS_ZK_REGION_OPENED (4, ExecutorType.MASTER_OPEN_REGION), > > > >If you follow RS_ZK_REGION_OPENED, you would see how the executor is used. > > > >On Thu, Jun 2, 2016 at 4:56 AM, WangYQ <wangyongqiang0...@163.com> wrote: > > > >> in hbase 0.98.10, class HMaster, method startServiceThread, hmaster > open a > >> thread poolwith type MASTER_OPEN_REGION, but this pool is not used in > any > >> place > >> can remove this. > >> > >> > >> thanks >