jmark99 commented on PR #3021:
URL: https://github.com/apache/accumulo/pull/3021#issuecomment-1279180572

   @ctubbsii , @dlmarion , would a change such as the following address your 
concerns?
   
   Replace:
   
   <pre>
       SortedSet<Path> logFiles = new TreeSet<>();
   </pre>
   
   within `getFiles` with:
   
   <pre>
       // Path::getName compares the last component of each Path value. In this 
case, the last
       // component should always have the format 'part-r-XXXXX.rf', where 
XXXXX are one-up values.
       SortedSet<Path> logFiles = new 
TreeSet<>(Comparator.comparing(Path::getName));
   </pre>
   
   Therefore, only comparing the 'part-r-xxxxx.rf' portion of the filenames.


-- 
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]

Reply via email to