keith-turner commented on code in PR #3480:
URL: https://github.com/apache/accumulo/pull/3480#discussion_r1239912604
##########
server/base/src/main/java/org/apache/accumulo/server/fs/VolumeUtil.java:
##########
@@ -162,11 +163,12 @@ public static TabletFiles
updateTabletVolumes(ServerContext context, ServiceLock
}
for (Entry<StoredTabletFile,DataFileValue> entry :
tabletFiles.datafiles.entrySet()) {
- String metaPath = entry.getKey().getMetaUpdateDelete();
Review Comment:
Yeah looking at this I was thinking what is the minimal amount of test and
code changes that is the most maintainable and ensures we have this correct.
It seems like we may need a lot of tests that verify lots of operations work
properly when there is a range present, I am not sure if there is anyway to
avoid that. We may want to open follow on issues for each test that is needed.
--
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]
