amahussein commented on pull request #3200: URL: https://github.com/apache/hadoop/pull/3200#issuecomment-909234801
> Thanks @amahussein for your works. I trigger Yetus and it seems works well. Failed unit test `TestDirectoryScanner ` looks not related to changes. AND verify this feature with pseudo cluster, it also seems work well. > Just notice that this PR include different cherry-pick commits together. I am not sure if it is traced gracefully when check in. > @brahmareddybattula do you have any idea here? If not I would like give my +1. Thanks @Hexiaoqiao for the review and the feedback. I appreciate it. I intentionally cherry-picked the commits related to the feature so that the PR would be complete. Otherwise, if we did separate PRs for each commit, branch-3.2 would be broken. I believe it should be okay to have multiple cherry-picks as long as the final merge would not squash them into one. -- 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]
