keith-turner commented on a change in pull request #2289:
URL: https://github.com/apache/accumulo/pull/2289#discussion_r716854295



##########
File path: 
server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/CompactableImpl.java
##########
@@ -262,7 +268,7 @@ void finishSelection(Set<StoredTabletFile> selected, 
boolean allSelected) {
       Preconditions.checkArgument(!selected.isEmpty());
       Preconditions.checkState(selectStatus == FileSelectionStatus.SELECTING);
       selectStatus = FileSelectionStatus.SELECTED;
-      selectedTime = System.currentTimeMillis();
+      selectedTime = getCurentTimeMillis();

Review comment:
       It would be better to use nano time.  Use of currentTimeMillis was 
present before this change.  I may update it in this change (if its simple to 
do), if not we can open an issue.




-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to