ctubbsii commented on a change in pull request #1670:
URL: https://github.com/apache/accumulo/pull/1670#discussion_r502615753



##########
File path: 
server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/DatafileManager.java
##########
@@ -458,6 +457,7 @@ StoredTabletFile 
bringMajorCompactionOnline(Set<StoredTabletFile> oldDatafiles,
         filesInUseByScans, newFile, compactionId, dfv,
         tablet.getTabletServer().getClientAddressString(), lastLocation,
         tablet.getTabletServer().getLock());
+    tablet.setLastCompactionID(compactionId);

Review comment:
       I believe there is a related comment a few lines above that can probably 
be removed, about `known consistency issue between minor and major compactions`




----------------------------------------------------------------
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:
[email protected]


Reply via email to