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

Reply via email to