apache / accumulo-testing

Apache Accumulo Testing
https://accumulo.apache.org
Apache License 2.0
15 stars 40 forks source link

Refactor logging in Verify for RW Bulk #254

Closed Manno15 closed 2 years ago

Manno15 commented 2 years ago

Closes #239.

Instead of throwing an exception at each possible failure, an error log is recorded instead. Currently, have it to break out of each loop but we could instead complete the loops and log each possible error. Need to do more testing there to see if that would be worthwhile.

Manno15 commented 2 years ago

https://github.com/apache/accumulo-testing/pull/254/commits/f5ff88e2130e57be61d7a4a25f54f4dbe7c3b064 addresses the comments.

Manno15 commented 2 years ago

Tests are failing due to a change in accumulo. I will look into fixing it tomorrow. This PR should be ready for review still.

EDIT: Build should be Fixed in accumulo-testing/256