Apache
/
hadoop
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

HADOOP-18172: Changed scope for isRootInternalDir/getRootFallbackLink for InodeTree (#4106) 

* HADOOP-18172: Change scope of InodeTree and its member methods to make them accessible from outside package.

Co-authored-by: Xing Lin <xinglin@linkedin.com>
This commit is contained in:
Xing Lin 2022-04-19 22:47:02 -07:00 committed by GitHub
parent ec0ff1dc04
commit 98b9c435f2
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/viewfs/InodeTree.java
View File

@ -61,7 +61,7 @@ import org.slf4j.LoggerFactory;
@InterfaceAudience.Private @InterfaceAudience.Private
@InterfaceStability.Unstable @InterfaceStability.Unstable
abstract class InodeTree<T> { public abstract class InodeTree<T> {
private static final Logger LOGGER = private static final Logger LOGGER =
LoggerFactory.getLogger(InodeTree.class.getName()); LoggerFactory.getLogger(InodeTree.class.getName());
@ -458,11 +458,11 @@ abstract class InodeTree<T> {
* there will be root to root mapping. So, root does not represent as * there will be root to root mapping. So, root does not represent as
* internalDir. * internalDir.
*/ */
protected boolean isRootInternalDir() { public boolean isRootInternalDir() {
return root.isInternalDir(); return root.isInternalDir();
} }
protected INodeLink<T> getRootFallbackLink() { public INodeLink<T> getRootFallbackLink() {
Preconditions.checkState(root.isInternalDir()); Preconditions.checkState(root.isInternalDir());
return rootFallbackLink; return rootFallbackLink;
} }
@ -742,7 +742,7 @@ abstract class InodeTree<T> {
* If the input pathname leads to an internal mount-table entry then * If the input pathname leads to an internal mount-table entry then
* the target file system is one that represents the internal inode. * the target file system is one that represents the internal inode.
*/ */
static class ResolveResult<T> { public static class ResolveResult<T> {
final ResultKind kind; final ResultKind kind;
final T targetFileSystem; final T targetFileSystem;
final String resolvedPath; final String resolvedPath;
@ -777,7 +777,7 @@ abstract class InodeTree<T> {
* @return ResolveResult which allows further resolution of the remaining path * @return ResolveResult which allows further resolution of the remaining path
* @throws IOException * @throws IOException
*/ */
ResolveResult<T> resolve(final String p, final boolean resolveLastComponent) public ResolveResult<T> resolve(final String p, final boolean resolveLastComponent)
throws IOException { throws IOException {
ResolveResult<T> resolveResult = null; ResolveResult<T> resolveResult = null;
String[] path = breakIntoPathComponents(p); String[] path = breakIntoPathComponents(p);
@ -957,7 +957,7 @@ abstract class InodeTree<T> {
* @return home dir value from mount table; null if no config value * @return home dir value from mount table; null if no config value
* was found. * was found.
*/ */
String getHomeDirPrefixValue() { public String getHomeDirPrefixValue() {
return homedirPrefix; return homedirPrefix;
} }
} }