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

   I have no issue with this superseding the #3771 and #3776.  As long as the 
test becomes stable. 
    (#3771 should be closed in favor of #3776 anyway.)
   
   We may want to consider a specific test that can create / test for the 
condition rather than hoping to hit it by chance in 
`ManagerRepairsDualAssignmentIT`  Changing the way killing and testing that it 
has been reported dead may be better as is done in #3776 - but that would mask 
hitting the thrift change.


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