-
Notifications
You must be signed in to change notification settings - Fork 8.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
HDFS-15916. DistCp: Backward compatibility: Distcp fails from Hadoop 3 to Hadoop 2 for snapshotdiff. #2863
Conversation
…3 to Hadoop 2 for snapshotdiff.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 (non-binding) based on Jira discussion.
💔 -1 overall
This message was automatically generated. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is good.
} catch (RpcNoSuchMethodException e) { | ||
// In case the server doesn't support getSnapshotDiffReportListing, | ||
// fallback to getSnapshotDiffReport. | ||
LOG.warn("Falling back to getSnapshotDiffReport {}", e.getMessage()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should use DFSClient.LOG instead because the rest of code logs using DFSClient.LOG.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeps, Raised Addendum PR:
#3056
…3 to Hadoop 2 for snapshotdiff. (apache#2863). Contributed by Ayush Saxena. Signed-off-by: Wei-Chiu Chuang <weichiu@apache.org> (cherry picked from commit c6539e3) Conflicts: hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/DistributedFileSystem.java
…3 to Hadoop 2 for snapshotdiff. (#2863). Contributed by Ayush Saxena. (#3054) Signed-off-by: Wei-Chiu Chuang <weichiu@apache.org> (cherry picked from commit c6539e3) Conflicts: hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/DistributedFileSystem.java Co-authored-by: Ayush Saxena <ayushsaxena@apache.org>
…3 to Hadoop 2 for snapshotdiff. (apache#2863). Contributed by Ayush Saxena. Signed-off-by: Wei-Chiu Chuang <weichiu@apache.org>
…few important for 7.1.7 SP1 Upstream JIRA ID: HDFS-15916. DistCp: Backward compatibility: Distcp fails from Hadoop 3 to Hadoop 2 for snapshotdiff. (apache#2863). Contributed by Ayush Saxena. (apache#3054) Signed-off-by: Wei-Chiu Chuang <weichiu@apache.org> (cherry picked from commit c6539e3) Co-authored-by: Ayush Saxena <ayushsaxena@apache.org> (cherry picked from commit 83d119f) Change-Id: Ie833e49e2baf532f48d8fbeb325f34c0f632664e (cherry picked from commit 1e8f156)
https://issues.apache.org/jira/browse/HDFS-15916