-
Notifications
You must be signed in to change notification settings - Fork 8.9k
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-16791. Add getEnclosingRoot() API to filesystem interface and im… #6262
HDFS-16791. Add getEnclosingRoot() API to filesystem interface and im… #6262
Conversation
…plementations (apache#6198) The enclosing root path is a common ancestor that should be used for temp and staging dirs as well as within encryption zones and other restricted directories. Contributed by Tom McCormick
@steveloughran Nearly clean cherry pick. Just a couple of things that exist in trunk but not in 3.3, so all deletes.
|
💔 -1 overall
This message was automatically generated. |
looks like the hadoop-rbf stuff didn't take. just did a local build of branch-3.3 and it is happy, so this probably really is a regression |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
@steveloughran This kind of seems like yetus issues to me now. I also locally saw one other failure locally which was the protobuf static import thats not in 3.3.
Im gonna try reverting the last commit (although it seems needed) to see if yetus likes it more |
This reverts commit 9603a9e.
After the revert I get building locally
|
💔 -1 overall
This message was automatically generated. |
@steveloughran Let me know if theres any concerns with the current failure, it seems transient / expected to me, but not 100% |
@goiri Thoughts? Does this backport look ok. The failure doesn't seem related to my backport right? |
LGTM. @steveloughran do you mind taking a look? |
I'm going to say unrelated. HDFS-16740/#4835 addresses flakiness in trunk, but it's not been backported. as usual: volunteers welcome. |
merged. If you are on common-dev you can see we've started a discussion about pushing out a branch-3.3 release... |
…plementations (#6198)
The enclosing root path is a common ancestor that should be used for temp and staging dirs as well as within encryption zones and other restricted directories.
Contributed by Tom McCormick
Description of PR
Cherry pick of #6198 (review)
How was this patch tested?
For code changes:
LICENSE
,LICENSE-binary
,NOTICE-binary
files?