milleruntime commented on pull request #2318: URL: https://github.com/apache/accumulo/pull/2318#issuecomment-964615695
So it turns out I was introducing the thrift dependency by adding these two imports: <pre> -import org.apache.thrift.TException; -import org.apache.thrift.transport.TTransportException; </pre> So I think what we have in the Monitor is a bit of a hack to get around this. By catching Exception, like we do when we get the ManagerClient, I can avoid importing the thrift exceptions and it doesn't complain about the dependency. -- 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]
