hadoop-hdfs-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Brandon Li (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (HDFS-4084) provide CLI support for allow and disallow snapshot on a directory
Date Tue, 23 Oct 2012 17:23:12 GMT

     [ https://issues.apache.org/jira/browse/HDFS-4084?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Brandon Li updated HDFS-4084:
-----------------------------

    Attachment: HDFS-4084.patch

{quote} When you are overriding, no need to add javadoc to the method if you plan on inheriting
changes from the super class. So javadoc for DistributedFileSystem methods you have added
can be deleted.{quote} 
Methods allowSnapshot and diskallowSnapshot are not inerited from FileSystem class, and thus
not override. Added javadoc for them.
   {quote}  Why are you adding @VisibleForTesting for snapshot related methods in FSNamesystem.
Also please proper javadoc to the methods.{quote} 
Removed, realized these two methods need to be public anyway.
    {quote} FSNamesystem.java is unnecessarily importing SnapshotInfo?{quote} 
Yes. removed.
    {quote} NamenodeRpcServer.java remove comment "Client Protocol" for overridden protocol
methods{quote} 
Done. Thanks.

                
> provide CLI support for allow and disallow snapshot on a directory
> ------------------------------------------------------------------
>
>                 Key: HDFS-4084
>                 URL: https://issues.apache.org/jira/browse/HDFS-4084
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>          Components: hdfs client, name-node, tools
>    Affects Versions: Snapshot (HDFS-2802)
>            Reporter: Brandon Li
>            Assignee: Brandon Li
>              Labels: needs-test
>         Attachments: HDFS-4084.patch, HDFS-4084.patch, HDFS-4084.patch
>
>
> To provide CLI support to allow snapshot, disallow snapshot on a directory.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Mime
View raw message