keith-turner commented on PR #6139: URL: https://github.com/apache/accumulo/pull/6139#issuecomment-3955034442
I have been looking into how to pull in some of the changes from #3262 and how to have a single thrift service and single server context, which are all related. The Highly Available Service that was removed in #5419 would be useful for having a single thrift server in the manager that can handle primary and non-primary RPC request. Looking back at #5419 one thing that was probably causing a lot of exceptions in the manager logs is that some thrift methods did not throw ThriftNotActiveServiceException. The highly available service code expected all thrift methods to throw this. When a thrift method does not declare that it will probably cause server side errors. Going to look into bringing the Highly Available Service code back in a stand along PR as that seems to be one pre req to this. Also going to change this PR to draft for now. One other thing I found RE the highly available service is that even though it was removed, the client side code for handling ThriftNotActiveServiceException is still there. -- 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]
