dlmarion commented on code in PR #3409:
URL: https://github.com/apache/accumulo/pull/3409#discussion_r1202859920
##########
core/src/main/java/org/apache/accumulo/core/clientImpl/TabletServerBatchReaderIterator.java:
##########
@@ -285,21 +285,22 @@ private ScanServerData binRanges(ClientTabletCache
clientTabletCache, List<Range
break;
} else {
// tried to only do table state checks when failures.size() ==
ranges.size(), however this
- // did
- // not work because nothing ever invalidated entries in the
tabletLocator cache... so even
- // though
- // the table was deleted the tablet locator entries for the deleted
table were not
- // cleared... so
- // need to always do the check when failures occur
+ // did not work because nothing ever invalidated entries in the
tabletLocator cache... so
+ // even though the table was deleted the tablet locator entries for
the deleted table
+ // were not cleared. So need to always do the check when failures occur
if (failures.size() >= lastFailureSize) {
context.requireNotDeleted(tableId);
context.requireNotOffline(tableId, tableName);
}
lastFailureSize = failures.size();
+ log.debug(
Review Comment:
Thanks, another artifact from debugging failed tests
--
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]
