[PR] Sleeps after eventual scan RPC failures [accumulo]

["keith-turner (via GitHub)" <gi...@apache.org>]

keith-turner opened a new pull request, #6315:
URL: https://github.com/apache/accumulo/pull/6315

   Fixes two problems. First the scan server client side plugins were not computing a sleep time based on observed errors.  This could cause aggressive retries with scans. Modified the provided plugins to compute this.
   
   Second the batch scanner code was not properly collecting the information needed by scan server client side plugin to know if errors happened.  The batch scanner code was not collecting information for all tablets, just a somewhat random subset of them.  Corrected it to collect for all tablets.  Also made the batch scanner code properly report failed tablets to the client side scan server plugin.
   
   Both changes together fix #6313.  Manually tested the batch scanner code changes by adding logs and running test to ensure the correct sleeps were happening. Also manually tested the scanner to ensure it was working correctly and sleeping as expected after errors.


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

Re: [PR] Sleeps after eventual scan RPC failures [accumulo]

["keith-turner (via GitHub)" <gi...@apache.org>]

keith-turner merged PR #6315:
URL: https://github.com/apache/accumulo/pull/6315


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