ddanielr commented on code in PR #5863:
URL: https://github.com/apache/accumulo/pull/5863#discussion_r2337219829
##########
server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/Tablet.java:
##########
@@ -1523,7 +1524,18 @@ public Optional<SplitComputations>
getSplitComputations() {
lastSplitComputation = new SoftReference<>(newComputation);
} catch (IOException e) {
lastSplitComputation.clear();
- log.error("Failed to compute split information from files " +
e.getMessage());
+ if (e.getClass().equals(FileNotFoundException.class)) {
+ Set<TabletFile> currentFiles = getDatafileManager().getFiles();
+ files.removeAll(currentFiles);
+ if (!files.isEmpty()) {
+ log.debug(
+ "Failed to compute split information. The following files have
most likely been garbage collected: {}",
+ files);
+ return Optional.empty();
+ }
Review Comment:
Reorganized this into a second catch block for FileNotFoundException
--
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]
