etseidl edited a comment on issue #1410: check that the WAL directory and log 
files can sync
URL: https://github.com/apache/accumulo/pull/1410#issuecomment-550451513
 
 
   @keith-turner you resolved the comments about multiple volumes, but that got 
me looking more deeply at how the volume chooser works.  Would it make more 
sense to add a getAllVolumesForPath() that returns an array of Volumes and then 
loop over those? Maybe pass in a scope as well?  I'm thinking not just about 
this PR, but looking ahead to the changes I want to make to add EC support at 
the table config level.

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

Reply via email to