Jtdellaringa commented on PR #7127: URL: https://github.com/apache/hadoop/pull/7127#issuecomment-2435771938
> Small comments then LGTM for adding the parameters. I assume this needs a separate commit for the functionality itself. Is OSS okay with merging this now or should we wait until we have both commits? This PR includes functionality - the change in private long init(List<DatanodeStorageReport> reports) method is where we filter out the allowed/excluded nodes from being targets or or sources -- 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: common-issues-unsubscr...@hadoop.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
