DomGarguilo commented on a change in pull request #2320:
URL: https://github.com/apache/accumulo/pull/2320#discussion_r747001226
##########
File path:
server/tserver/src/main/java/org/apache/accumulo/tserver/TabletServer.java
##########
@@ -791,6 +791,27 @@ public void run() {
}
}, 0, 5000, TimeUnit.MILLISECONDS);
+
ThreadPools.createGeneralScheduledExecutorService(aconf).scheduleWithFixedDelay(()
-> {
+ final SortedMap<KeyExtent,Tablet> onlineTabletsSnapshot =
onlineTablets.snapshot();
+ final SortedMap<KeyExtent,Pair<Long,TabletMetadata>>
tabletValidationInfo = new TreeMap<>();
+
+ // gather update counters and metadata for all tablets
+ for (var entry : onlineTabletsSnapshot.entrySet()) {
+ KeyExtent keyExtent = entry.getKey();
+ Long counter = entry.getValue().getUpdateCounter();
+ TabletMetadata tm = getContext().getAmple().readTablet(keyExtent,
+ TabletMetadata.ColumnType.FILES, TabletMetadata.ColumnType.LOGS,
+ TabletMetadata.ColumnType.ECOMP,
TabletMetadata.ColumnType.PREV_ROW);
Review comment:
The most recent changes (4f282f4) use readTablets(). However, I am not
familiar with this method so I'm not sure the difference this might make
regarding the data levels. @keith-turner
--
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]
