[
https://issues.apache.org/jira/browse/HADOOP-18172?focusedWorklogId=758162&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-758162
]
ASF GitHub Bot logged work on HADOOP-18172:
-------------------------------------------
Author: ASF GitHub Bot
Created on: 18/Apr/22 23:08
Start Date: 18/Apr/22 23:08
Worklog Time Spent: 10m
Work Description: xinglin commented on code in PR #4106:
URL: https://github.com/apache/hadoop/pull/4106#discussion_r852473290
##########
hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/viewfs/InodeTree.java:
##########
@@ -458,11 +458,11 @@ private boolean hasFallbackLink() {
* there will be root to root mapping. So, root does not represent as
* internalDir.
*/
- protected boolean isRootInternalDir() {
+ public boolean isRootInternalDir() {
Review Comment:
We need to access just these two functions for our current use case. We can
make more functions public is they are needed in the future.
Issue Time Tracking
-------------------
Worklog Id: (was: 758162)
Time Spent: 2h 50m (was: 2h 40m)
> Change scope of getRootFallbackLink for InodeTree to make them accessible
> from outside package
> ----------------------------------------------------------------------------------------------
>
> Key: HADOOP-18172
> URL: https://issues.apache.org/jira/browse/HADOOP-18172
> Project: Hadoop Common
> Issue Type: Improvement
> Reporter: Xing Lin
> Assignee: Xing Lin
> Priority: Minor
> Labels: pull-request-available
> Time Spent: 2h 50m
> Remaining Estimate: 0h
>
> Sometimes, we need to access rootFallBackLink in InodeTree from another
> package. One such case is we extend from ViewFileSystem but want to put the
> new filesystem in org.apache.hadoop.fs package, instead of
> org.apache.hadoop.fs.viewfs package. As a result, we need make them public,
> similar as what we did for getMountPoints() in HADOOP-18100.
--
This message was sent by Atlassian Jira
(v8.20.1#820001)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
