WangXin

Results 7 comments of WangXin

> This is a nice addition. Is it possible add test for this condition in TestRegionNormalizerWorker? It is ok.

@wchevreuil hello, If you have time, please take a review again.

> Please fix the spoltess issue? It is ok. Next time, I'll pay attention. Thanks.

Ok, I understand Now, the create time of the peer needs to be recorded separately on the zk node for persistence, so I need to create a new node under...

> > Ok, I understand Now, the create time of the peer needs to be recorded separately on the zk node for persistence, so I need to create a new...

` long timeout = EnvironmentEdgeManager.currentTime() + this.shipEditsTimeout;` `this.shipEditsTimeout` = this.conf.getInt(HConstants.REPLICATION_SOURCE_SHIPEDITS_TIMEOUT, HConstants.REPLICATION_SOURCE_SHIPEDITS_TIMEOUT_DFAULT);` This timeout is configurable. In this case, is there any other better way to solve the problem? @Apache9

> I prefer we change the config name to something like 'compaction.skip-merging-compact', to indicate that this config is used to skip the real compaction operation. > > As even if...