szetszwo opened a new pull request, #11294: URL: https://github.com/apache/ozone/pull/11294
## What changes were proposed in this pull request? We may simplify XceiverClientShortCircuit in order to support concurrent access: - Add a name field and do not use domainSocket for printing messages. - Refactor timer to an inner class and make all its methods synchronized. - Simplify sendCommandWithTraceID, which currently calls CompletableFuture.get() to obtain the responseProto and then use it to create another CompletableFuture -- just use the first CompletableFuture. - Change requestSent and responseReceived, which is just for printing debug messages, to AtomicLong. - Remove RequestEntry.sentTimeNs which is just for printing debug messages. Print it in a different location. - Remove RequestEntry.timerTask -- use CompletableFuture.whenComplete(..) - Convert the prefix field to a local variable. - Remove the unused the config field. ## What is the link to the Apache JIRA HDDS-16517 ## How was this patch tested? By updating existing tests. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
