nandakumar131 commented on issue #313: HDDS-2242. Avoid unnecessary rpc needed to discover the pipeline leader. URL: https://github.com/apache/hadoop-ozone/pull/313#issuecomment-565333775 > > The code will become a lot simpler if the leader node in `Pipeline` is represented as `DatanodeDetails` instead of just `UUID`. > > Hi, @nandakumar131 this was changed based on your suggestion in HDDS-1868 :-) > Although it isn't affecting performance, can we refactor later, this patch was meant as downstream work and I was hoping not to change proto structure for it. @swagle Even in HDDS-1868 the suggestion was to use `DatanodeDetails` instead of just `UUID`. We can do it in a follow-up jira. I will merge this shortly.
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: ozone-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: ozone-issues-h...@hadoop.apache.org
