Manno15 commented on issue #2327: URL: https://github.com/apache/accumulo/issues/2327#issuecomment-953334759
> I am not sure how the Manager can still accept RPC calls after being shut down but that's what the test failing in ManagerApiIT is testing. There are two calls to expectPermissionSuccess() made after shutting down, the first one succeeds but then it will hang indefinitely on the second. If you remove one of them, the test passes. So it seems like when the expectPermissionSuccess() succeeds, it closes things for good, which is odd since they both call the same code and should both be executing the finally in ManagerClient. That is what I found too. Tracing through the operation, manager doesn't finish shutting down until after the second one starts. Then it will run into a connection refused exception and then it will notice there are no managers left but still keep retrying. I am not sure what the original intention of this test was but it does seem with the changes in #2303 either things shut down faster or it takes longer to run through the operation fully. The test originally expected both expectPermissionSuccess calls to start and finish on the same manager before either one fully closed it. Not sure why though since the test seems to only care about the permissions of the user. -- 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]
