accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [accumulo] foster33 opened a new pull request #2180: Update CheckForMetadataProblems Utility Output
Date Thu, 24 Jun 2021 15:40:25 GMT

foster33 opened a new pull request #2180:
URL: https://github.com/apache/accumulo/pull/2180


   Fixes #2166 
   
   This PR updates the output of the CheckForMetadataProblems utility. It previously included
a mix of table ids and names.
   
   The output now offers more information and is much more easily read.
   
   Some changes and additions that are included:
   
   - `checkTable` function now accepts a String tableName 
   - Uses `Tables.getTableID()` and `Tables.getTableName()` to get information about the specific
tables
   - Places the client scanner inside of the try block in `checkMetadataAndRootTableEntries()`
   - Updates potential error messages that were not shown in the output in the initial ticket.
   
   During my testing I did notice that there is an extra line that shows some log info about
the path to accumulo.properties that gets displayed before the expected output. I am not sure
if this is a big issue that it is displayed, but if there is a way to not have it displayed
that would probably be best.


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



Mime
View raw message