sodonnel opened a new pull request, #4413: URL: https://github.com/apache/ozone/pull/4413
## What changes were proposed in this pull request? In order to keep a consistent count of the number of pending replicate and delete commands, ReplicationManager should send commands to the datanode via the nodeManager.addDatanodeCommand() method rather than publishing them to the event queue. At the moment, the commands goto the event queue, where they are picked up by NodeManager and added to the datanode command queue. As we are using the counts in the command queue to determine how many commands are pending, the indirection provided by the event queue makes it difficult to keep a consistent count without excessive effort. There is also no further processing done on the commands in the event queue, they simply go RM -> Event Queue -> Node Manager -> Command Queue. This change makes RM send the commands to nodeManager directly. ## What is the link to the Apache JIRA https://issues.apache.org/jira/browse/HDDS-8187 ## How was this patch tested? Modified existing tests to ensure nodeManager is getting called rather than eventQueue. -- 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: issues-unsubscr...@ozone.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@ozone.apache.org For additional commands, e-mail: issues-h...@ozone.apache.org
