[GitHub] [accumulo] ctubbsii commented on issue #1539: Fixes #1525 - remove lines added in b67d3f9 in VolumeUtil

[GitBox <gi...@apache.org>]

ctubbsii commented on issue #1539: Fixes #1525 - remove lines added in b67d3f9 in VolumeUtil
URL: https://github.com/apache/accumulo/pull/1539#issuecomment-593522604
 
 
   > This is odd, because it is passing in master with this change. Comparing the IT across the two versions is rather difficult though since there have been so many changes.
   > 
   > I am also not sure why the original change doesn't work. If you look at the code that calls this method [here](https://github.com/apache/accumulo/blob/94844b3ef88bc9e02b5cb5c8eeff81411ac8221d/server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/Tablet.java#L344), the `tabletPaths` object should be initialized with all of the entries.
   
   The TabletFiles that is initialized there isn't the one that gets returned from this method. The one returned from this method is initialized at https://github.com/apache/accumulo/blob/94844b3ef88bc9e02b5cb5c8eeff81411ac8221d/server/base/src/main/java/org/apache/accumulo/server/fs/VolumeUtil.java#L151

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services