dlmarion commented on PR #4879: URL: https://github.com/apache/accumulo/pull/4879#issuecomment-2419315665
Can/should we remove the logs feature from the Monitor in a 2.1 patch release? Should we just tell users not to configure the Monitor appender in the 2.1.4 release notes and warn them that if they do they run the risk of overloading the Monitor host with TCP connections? If we want/need to retain the logs feature in the Monitor, then I think the possible solutions are either implement `http/2+tls` or implement the Thrift RPC mechanism described in #4982. FWIW, it looks like http/2 without TLS is still a thing, they just removed the ability to discover that from the protocol. If both the server and client know via configuration that they are to speak http/2 without tls, then it should still work. -- 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]
