You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2022/01/19 18:21:16 UTC

[GitHub] [accumulo] ctubbsii commented on pull request #1259: update du command to use hdfs iterator

ctubbsii commented on pull request #1259:
URL: https://github.com/apache/accumulo/pull/1259#issuecomment-1016742918


   > I think this can be merged for 2.1. @keith-turner @ctubbsii please let me know within the next week if you have any objections.
   
   I think it's ready, but I still had the unresolved comment about making sure that it only returns files, and not directories, which could be confirmed through some experimentation. I suggest doing a quick check to make sure the recursive file listing works as expected, only returning files and not directories before merging. But otherwise, I think it's good to go.


-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org