You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2021/09/27 16:26:33 UTC

[GitHub] [accumulo] keith-turner commented on a change in pull request #2289: Adds unit test for compaction file tracking

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 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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org