dlmarion commented on PR #4715: URL: https://github.com/apache/accumulo/pull/4715#issuecomment-2245023645
>Here is an update to the current state of things. > ... > Follow on Work: > * Add support for using a random port with gRPC when setting the port to 0. > * Explore supporting kerberos and ssl w/ grpc > * Explore how to implement tracing with gRPC. > * Investigate error and exception handling with gRPC. > * Investigate any other properties that need to be customized for the gRPC server and clients. I feel like we put the cart before the horse here. These items tagged for follow-on work are actually foundational things that, IMO, should be designed and tested first. I think the work that you have done here is useful to show that grpc can be used, but I'm not convinced that this is the starting point for the introduction of grpc into the codebase. -- 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]
