[jira] [Updated] (HBASE-24581) Skip compaction request/check for replica regions at the early stage.

["Huaxiang Sun (Jira)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/HBASE-24581?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Huaxiang Sun updated HBASE-24581:
---------------------------------
    Summary: Skip compaction request/check for replica regions at the early stage.  (was: Skip compaction request/check for replica regions at the )

> Skip compaction request/check for replica regions at the early stage.
> ---------------------------------------------------------------------
>
>                 Key: HBASE-24581
>                 URL: https://issues.apache.org/jira/browse/HBASE-24581
>             Project: HBase
>          Issue Type: Improvement
>          Components: read replicas
>    Affects Versions: 2.3.0
>            Reporter: Huaxiang Sun
>            Assignee: Huaxiang Sun
>            Priority: Major
>
> I found that in certain cases replica regions can trigger compaction, one example as follows, need to check all places to avoid compaction for replica regions.
> {code:java}
> @Override
> public void postOpenDeployTasks(final PostOpenDeployContext context) throws IOException {
>   HRegion r = context.getRegion();
>   long openProcId = context.getOpenProcId();
>   long masterSystemTime = context.getMasterSystemTime();
>   rpcServices.checkOpen();
>   LOG.info("Post open deploy tasks for {}, openProcId={}, masterSystemTime={}",
>     r.getRegionInfo().getRegionNameAsString(), openProcId, masterSystemTime);
>   // Do checks to see if we need to compact (references or too many files)
>   // TODO: SHX, do not do this for replica regions? Otherwise, it is going to lost data locality for primary regions.
>   for (HStore s : r.stores.values()) {
>     if (s.hasReferences() || s.needsCompaction()) {
>       this.compactSplitThread.requestSystemCompaction(r, s, "Opening Region");
>     }
>   }
>  {code}



--
This message was sent by Atlassian Jira
(v8.3.4#803005)