[GitHub] [accumulo] EdColeman commented on pull request #3771: update test to handle connect error when tserver killed

["EdColeman (via GitHub)" <gi...@apache.org>]

EdColeman commented on PR #3771:
URL: https://github.com/apache/accumulo/pull/3771#issuecomment-1732138039

   Not sure if this strengthens the case for improving the error handling in `TabletStateStore` - but in the test, the bad server scan sees the communication error - but it does not reach the threshold to mark the server as bad (it takes 3 failed attempts).  It seems that depending on which server is used by `TabletStateStore` the test passes if it picks the "good" server.  If it picks the dead, but not declared dead server, the socket closed / EOF exception triggers the test failure.
   
   It seems that trying to talk to a recently deceased server should be an exceptional condition that we should handle correctly rather than working around it in the test.


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