[
https://issues.apache.org/jira/browse/HBASE-17172?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15692632#comment-15692632
]
Jingcheng Du commented on HBASE-17172:
--------------------------------------
Thanks Huaxiang!
bq. users may still choose to disable mob compaction chore and run mob
compaction manually at scheduled maintenance.
Right, how about to run minor compaction instead? It doesn't make sense to run
major mob compaction periodically. Mob is designed to reduce the IO
amplification during compaction. Major compaction will break this.
bq. To keep delete marker in hbase files in mob-enabled cf is one way to avoid
.del files, the concern is that it is inconsistent with non-mob cfs (maybe this
can be provided as option through config?).
Hmm. If the .del is not a performance killer, we don't need this. I reviewed
the code, I think the .del files is not the reason of the slow compaction,
major compaction itself is.
bq. With the current major mob compaction, these .del files will be included in
compacting of files for other regions which is not necessary.
Right, it is not necessary. To split them by regions is an good choice. But is
this necessary if .del file didn't impact the compaction performance badly?
> Optimize major mob compaction with _del files
> ---------------------------------------------
>
> Key: HBASE-17172
> URL: https://issues.apache.org/jira/browse/HBASE-17172
> Project: HBase
> Issue Type: Improvement
> Components: mob
> Affects Versions: 2.0.0
> Reporter: huaxiang sun
> Assignee: huaxiang sun
>
> Today, when there is a _del file in mobdir, with major mob compaction, every
> mob file will be recompacted, this causes lots of IO and slow down major mob
> compaction (may take months to finish). This needs to be improved. A few
> ideas are:
> 1) Do not compact all _del files into one, instead, compact them based on
> groups with startKey as the key. Then use firstKey/startKey to make each mob
> file to see if the _del file needs to be included for this partition.
> 2). Based on the timerange of the _del file, compaction for files after that
> timerange does not need to include the _del file as these are newer files.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
