[
https://issues.apache.org/jira/browse/HDFS-14228?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16752372#comment-16752372
]
Hudson commented on HDFS-14228:
-------------------------------
SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #15829 (See
[https://builds.apache.org/job/Hadoop-trunk-Commit/15829/])
HDFS-14228. Incorrect getSnapshottableDirListing() javadoc. Contributed
(weichiu: rev 45caeee6cfcf1ae3355cd880402159cf31e94a8a)
* (edit)
hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/DistributedFileSystem.java
* (edit)
hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/protocol/ClientProtocol.java
* (edit)
hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/FSNamesystem.java
> Incorrect getSnapshottableDirListing() javadoc
> ----------------------------------------------
>
> Key: HDFS-14228
> URL: https://issues.apache.org/jira/browse/HDFS-14228
> Project: Hadoop HDFS
> Issue Type: Bug
> Components: snapshots
> Affects Versions: 2.1.0-beta
> Reporter: Wei-Chiu Chuang
> Assignee: Dinesh Chitlangia
> Priority: Major
> Fix For: 3.0.4, 3.1.2, 3.3.0, 3.2.1
>
> Attachments: HDFS-14228.00.patch
>
>
> The Javadoc for {{DistributedFileSystem#getSnapshottableDirListing()}} is not
> consistent with {{FSNamesystem#getSnapshottableDirListing()}}
> {code:title=ClientProtocol#getSnapshottableDirListing()}
> /**
> * Get listing of all the snapshottable directories.
> *
> * @return Information about all the current snapshottable directory
> * @throws IOException If an I/O error occurred
> */
> @Idempotent
> @ReadOnly(isCoordinated = true)
> SnapshottableDirectoryStatus[] getSnapshottableDirListing()
> throws IOException;
> {code}
> {code:title=DistributedFileSystem#getSnapshottableDirListing()}
> /**
> * @return All the snapshottable directories
> * @throws IOException
> */
> public SnapshottableDirectoryStatus[] getSnapshottableDirListing()
> {code}
> But the implementation at NameNode side is:
> {code:title=FSNamesystem#getSnapshottableDirListing()}
> /**
> * Get the list of snapshottable directories that are owned
> * by the current user. Return all the snapshottable directories if the
> * current user is a super user.
> * @return The list of all the current snapshottable directories
> * @throws IOException
> */
> public SnapshottableDirectoryStatus[] getSnapshottableDirListing()
> {code}
> That is, if this method is called by a non-super user, it does not return all
> snapshottable directories. File this jira to get this corrected to avoid
> confusion.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
