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 2021/09/13 19:19:51 UTC

[GitHub] [accumulo-testing] keith-turner commented on pull request #152: Added check for existing output directory in /bin/cingest verify

keith-turner commented on pull request #152:
URL: https://github.com/apache/accumulo-testing/pull/152#issuecomment-918502301


   >  It looks like the files produced by running verify are empty temporary files that don't have any use.
   
   When the verify job does not detect any missing data the files are empty.  When missing data is detected, the files contain information about the missing data that is useful in debugging the problem of why Accumulo lost data.
   
   >  To me it seems like the best course of action would to be to delete the directory before each run without a prompt to the user.
   
   I like that behavior.


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