keith-turner commented on code in PR #4287:
URL: https://github.com/apache/accumulo/pull/4287#discussion_r1503504875
##########
server/manager/src/main/java/org/apache/accumulo/manager/TabletGroupWatcher.java:
##########
@@ -680,7 +682,19 @@ public void run() {
iter = store.iterator(tableMgmtParams);
var tabletMgmtStats = manageTablets(iter, tableMgmtParams,
currentTServers, true);
- lookForTabletsNeedingVolReplacement =
tabletMgmtStats.totalVolumeReplacements != 0;
+
+ // If currently looking for volume replacements, determine if the next
round needs to look.
+ if (lookForTabletsNeedingVolReplacement) {
+ // Continue to look for tablets needing volume replacement if there
was an error
Review Comment:
The formatter wrapped this oddly. When it does this I usually do the
following to fix it.
1. Collapse the long comment to a single line
2. run the mvn build to reformat the code
The above usually produces a much more nicely formatted comment.
--
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]
