sadanand48 commented on PR #5364: URL: https://github.com/apache/hadoop/pull/5364#issuecomment-1470839515
> I'm worried that removing a constant and the ability to set a new sync class will break something. Thanks @steveloughran for reviewing this PR. This constant is not present in the hadoop code but was added as part of a previous commit of this same PR, I'm no longer using that approach of setting the distcp.sync.class that I proposed earlier , instead just assigning the correct filesystem dynamically at run time did the job for me. -- 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