hadoop-hdfs-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Srikanth Sundarrajan (JIRA)" <j...@apache.org>
Subject [jira] Updated: (HDFS-1012) documentLocation attribute in LdapEntry for HDFSProxy isn't specific to a cluster
Date Wed, 07 Apr 2010 00:28:33 GMT

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

Srikanth Sundarrajan updated HDFS-1012:
---------------------------------------

    Attachment: HDFS-1012.patch

{quote}
    *  HDFS_PATH_PATTERN would not work if the port number is omitted.
    * I suggest to store the namenode string in init(..). Then, conf could become a local
variable in init(..)
    * The current test seems not covering all of the cases.
{quote}

Nicholas, Thanks for your review comments. Revised patch uploaded with the following changes

1. HDFS_PATH_PATTERN replaced from ^hdfs://([\\w\\-]+(\\.)?)+:\\d+ to (^hdfs://([\\w\\-]+(\\.)?)+:\\d+|^hdfs://([\\w\\-]+(\\.)?)+)
to support cases with no port #

2. conf.get("fs.default.name") is retrieved once in init and namenode url stored for future
reference and access. This doesn't change within the filter while webapp context is running

3. Three additional test cases have been added
    * Case for allowing a request with a valid unqualified documentLocation (path)
    * Case for allowing a request with a valid qualified documentLocation
    * Case for rejecting request where documentLocation doesn't contain a valid path for the
cluster in question 

------ Output from test-patch & test-contrib ------

     [exec] +1 overall.  
     [exec] 
     [exec]     +1 @author.  The patch does not contain any @author tags.
     [exec] 
     [exec]     +1 tests included.  The patch appears to include 5 new or modified tests.
     [exec] 
     [exec]     +1 javadoc.  The javadoc tool did not generate any warning messages.
     [exec] 
     [exec]     +1 javac.  The applied patch does not increase the total number of javac compiler
warnings.
     [exec] 
     [exec]     +1 findbugs.  The patch does not introduce any new Findbugs warnings.
     [exec] 
     [exec]     +1 release audit.  The applied patch does not increase the total number of
release audit warnings.


   [cactus] Running org.apache.hadoop.hdfsproxy.TestAuthorizationFilter
   [cactus] Tomcat 5.x started on port [30300]
   [cactus] Tests run: 4, Failures: 0, Errors: 0, Time elapsed: 0.972 sec
   [cactus] Running org.apache.hadoop.hdfsproxy.TestLdapIpDirFilter
   [cactus] Tests run: 2, Failures: 0, Errors: 0, Time elapsed: 0.789 sec
   [cactus] Running org.apache.hadoop.hdfsproxy.TestProxyFilter
   [cactus] Tests run: 2, Failures: 0, Errors: 0, Time elapsed: 0.867 sec
   [cactus] Running org.apache.hadoop.hdfsproxy.TestProxyForwardServlet
   [cactus] Tests run: 2, Failures: 0, Errors: 0, Time elapsed: 0.508 sec
   [cactus] Running org.apache.hadoop.hdfsproxy.TestProxyUtil
   [cactus] Tests run: 1, Failures: 0, Errors: 0, Time elapsed: 1.84 sec
   [cactus] Tomcat 5.x is stopping...
   [cactus] Tomcat 5.x is stopped

test:

BUILD SUCCESSFUL
Total time: 4 minutes 8 seconds


> documentLocation attribute in LdapEntry for HDFSProxy isn't specific to a cluster
> ---------------------------------------------------------------------------------
>
>                 Key: HDFS-1012
>                 URL: https://issues.apache.org/jira/browse/HDFS-1012
>             Project: Hadoop HDFS
>          Issue Type: Improvement
>          Components: contrib/hdfsproxy
>    Affects Versions: 0.20.1, 0.20.2, 0.21.0, 0.22.0
>            Reporter: Srikanth Sundarrajan
>            Assignee: Srikanth Sundarrajan
>             Fix For: 0.22.0
>
>         Attachments: HDFS-1012-bp-y20.patch, HDFS-1012-bp-y20s.patch, HDFS-1012.patch,
HDFS-1012.patch
>
>
> List of allowed document locations accessible through HDFSProxy isn't specific to a cluster.
LDAP entries can include the name of the cluster to which the path belongs to have better
control on which clusters/paths are accessible through HDFSProxy by the user.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message