anoopsjohn commented on a change in pull request #1784:
URL: https://github.com/apache/hbase/pull/1784#discussion_r431027896
##########
File path:
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HStore.java
##########
@@ -1921,9 +1921,15 @@ public boolean shouldPerformMajorCompaction() throws
IOException {
// If we're enqueuing a major, clear the force flag.
this.forceMajor = this.forceMajor && !request.isMajor();
- // Set common request properties.
- // Set priority, either override value supplied by caller or from
store.
- request.setPriority((priority != Store.NO_PRIORITY) ? priority :
getCompactPriority());
+ if (request.isAfterSplit()) {
Review comment:
@virajjasani One Q.
Rather than handling at this layer by checking whether the call is after a
split or not, can we set a higher priority on the compact request, created
after the split op? So then we dont need to set with these split status etc
and handle specially at this layer.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
