dlmarion commented on PR #5668:
URL: https://github.com/apache/accumulo/pull/5668#issuecomment-3019830111

   > Would be really nice to add some tests to TabletManagementIteratorIT to 
cover a few cases and ensure that the tablets that are ok are returned and the 
ones that have illegal metadata are reported. Maybe something like the 
following?
   > 
   > Maybe something like the following
   > 
   > 1. ok tablet that needs split
   > 2. tablet w/ missing end row
   > 3. ok tablet
   > 4. tablet w/ invalid json in the file column
   > 5. ok tablet
   > 6. tablet w/ valid json w/ a range that does not overlap that tablet (not 
sure this will cause a failure)
   > 7. What other busted tablet metadata situations could we cover?
   
   I added a test in TabletManagementIteratorIT for a tablet that does not have 
a prev end row. There is a metadata constraint check (and test) for a file with 
invalid json (code 3100), so I'm not sure that I would be able to introduce 
that error.


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