You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-issues@hadoop.apache.org by "Devaraj Das (JIRA)" <ji...@apache.org> on 2010/06/08 00:06:11 UTC

[jira] Created: (HADOOP-6813) Add a new newInstance method in FileSystem that takes a "user" as argument

Add a new newInstance method in FileSystem that takes a "user" as argument
--------------------------------------------------------------------------

                 Key: HADOOP-6813
                 URL: https://issues.apache.org/jira/browse/HADOOP-6813
             Project: Hadoop Common
          Issue Type: Improvement
          Components: fs
            Reporter: Devaraj Das
            Assignee: Devaraj Das
             Fix For: 0.22.0


In order to implement HDFS-1000 for trunk, I need to have a newInstance API in FileSystem that takes a "user" as an argument.

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


[jira] Updated: (HADOOP-6813) Add a new newInstance method in FileSystem that takes a "user" as argument

Posted by "Eli Collins (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HADOOP-6813?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Eli Collins updated HADOOP-6813:
--------------------------------

    Fix Version/s: 0.21.0
         Priority: Blocker  (was: Major)

Flagging for Tom to commit to 21.

> Add a new newInstance method in FileSystem that takes a "user" as argument
> --------------------------------------------------------------------------
>
>                 Key: HADOOP-6813
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6813
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: fs
>            Reporter: Devaraj Das
>            Assignee: Devaraj Das
>            Priority: Blocker
>             Fix For: 0.21.0, 0.22.0
>
>         Attachments: h-6813.patch
>
>
> In order to implement HDFS-1000 for trunk, I need to have a newInstance API in FileSystem that takes a "user" as an argument.

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


[jira] Commented: (HADOOP-6813) Add a new newInstance method in FileSystem that takes a "user" as argument

Posted by "Hudson (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HADOOP-6813?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12877977#action_12877977 ] 

Hudson commented on HADOOP-6813:
--------------------------------

Integrated in Hadoop-Common-trunk-Commit #297 (See [http://hudson.zones.apache.org/hudson/job/Hadoop-Common-trunk-Commit/297/])
    Move entries for HADOOP-6769 and HADOOP-6813 to 0.21 section of CHANGES.txt


> Add a new newInstance method in FileSystem that takes a "user" as argument
> --------------------------------------------------------------------------
>
>                 Key: HADOOP-6813
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6813
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: fs
>            Reporter: Devaraj Das
>            Assignee: Devaraj Das
>            Priority: Blocker
>             Fix For: 0.21.0
>
>         Attachments: h-6813.patch
>
>
> In order to implement HDFS-1000 for trunk, I need to have a newInstance API in FileSystem that takes a "user" as an argument.

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


[jira] Commented: (HADOOP-6813) Add a new newInstance method in FileSystem that takes a "user" as argument

Posted by "Chris Douglas (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HADOOP-6813?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12876481#action_12876481 ] 

Chris Douglas commented on HADOOP-6813:
---------------------------------------

This issue went from created to committed in less than an hour. Changes to the {{FileSystem}} API should require more notice and explanation than was provided in this JIRA, though this looks like a reasonable convenience method to me.

> Add a new newInstance method in FileSystem that takes a "user" as argument
> --------------------------------------------------------------------------
>
>                 Key: HADOOP-6813
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6813
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: fs
>            Reporter: Devaraj Das
>            Assignee: Devaraj Das
>             Fix For: 0.22.0
>
>         Attachments: h-6813.patch
>
>
> In order to implement HDFS-1000 for trunk, I need to have a newInstance API in FileSystem that takes a "user" as an argument.

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


[jira] Updated: (HADOOP-6813) Add a new newInstance method in FileSystem that takes a "user" as argument

Posted by "Devaraj Das (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HADOOP-6813?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Devaraj Das updated HADOOP-6813:
--------------------------------

    Status: Patch Available  (was: Open)

> Add a new newInstance method in FileSystem that takes a "user" as argument
> --------------------------------------------------------------------------
>
>                 Key: HADOOP-6813
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6813
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: fs
>            Reporter: Devaraj Das
>            Assignee: Devaraj Das
>             Fix For: 0.22.0
>
>         Attachments: h-6813.patch
>
>
> In order to implement HDFS-1000 for trunk, I need to have a newInstance API in FileSystem that takes a "user" as an argument.

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


[jira] Commented: (HADOOP-6813) Add a new newInstance method in FileSystem that takes a "user" as argument

Posted by "Hudson (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HADOOP-6813?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12876475#action_12876475 ] 

Hudson commented on HADOOP-6813:
--------------------------------

Integrated in Hadoop-Common-trunk-Commit #285 (See [http://hudson.zones.apache.org/hudson/job/Hadoop-Common-trunk-Commit/285/])
    HADOOP-6813. Add a new newInstance method in FileSystem that takes a user as argument


> Add a new newInstance method in FileSystem that takes a "user" as argument
> --------------------------------------------------------------------------
>
>                 Key: HADOOP-6813
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6813
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: fs
>            Reporter: Devaraj Das
>            Assignee: Devaraj Das
>             Fix For: 0.22.0
>
>         Attachments: h-6813.patch
>
>
> In order to implement HDFS-1000 for trunk, I need to have a newInstance API in FileSystem that takes a "user" as an argument.

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


[jira] Updated: (HADOOP-6813) Add a new newInstance method in FileSystem that takes a "user" as argument

Posted by "Devaraj Das (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HADOOP-6813?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Devaraj Das updated HADOOP-6813:
--------------------------------

    Attachment: h-6813.patch

Straightforward patch. 

> Add a new newInstance method in FileSystem that takes a "user" as argument
> --------------------------------------------------------------------------
>
>                 Key: HADOOP-6813
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6813
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: fs
>            Reporter: Devaraj Das
>            Assignee: Devaraj Das
>             Fix For: 0.22.0
>
>         Attachments: h-6813.patch
>
>
> In order to implement HDFS-1000 for trunk, I need to have a newInstance API in FileSystem that takes a "user" as an argument.

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


[jira] Updated: (HADOOP-6813) Add a new newInstance method in FileSystem that takes a "user" as argument

Posted by "Tom White (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HADOOP-6813?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Tom White updated HADOOP-6813:
------------------------------

     Release Note: I've just committed this to 0.21.
    Fix Version/s:     (was: 0.22.0)

> Add a new newInstance method in FileSystem that takes a "user" as argument
> --------------------------------------------------------------------------
>
>                 Key: HADOOP-6813
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6813
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: fs
>            Reporter: Devaraj Das
>            Assignee: Devaraj Das
>            Priority: Blocker
>             Fix For: 0.21.0
>
>         Attachments: h-6813.patch
>
>
> In order to implement HDFS-1000 for trunk, I need to have a newInstance API in FileSystem that takes a "user" as an argument.

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


[jira] Commented: (HADOOP-6813) Add a new newInstance method in FileSystem that takes a "user" as argument

Posted by "Hudson (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HADOOP-6813?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12878255#action_12878255 ] 

Hudson commented on HADOOP-6813:
--------------------------------

Integrated in Hadoop-Common-trunk #364 (See [http://hudson.zones.apache.org/hudson/job/Hadoop-Common-trunk/364/])
    Move entries for HADOOP-6769 and HADOOP-6813 to 0.21 section of CHANGES.txt


> Add a new newInstance method in FileSystem that takes a "user" as argument
> --------------------------------------------------------------------------
>
>                 Key: HADOOP-6813
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6813
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: fs
>            Reporter: Devaraj Das
>            Assignee: Devaraj Das
>            Priority: Blocker
>             Fix For: 0.21.0
>
>         Attachments: h-6813.patch
>
>
> In order to implement HDFS-1000 for trunk, I need to have a newInstance API in FileSystem that takes a "user" as an argument.

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


[jira] Commented: (HADOOP-6813) Add a new newInstance method in FileSystem that takes a "user" as argument

Posted by "Hudson (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HADOOP-6813?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12876633#action_12876633 ] 

Hudson commented on HADOOP-6813:
--------------------------------

Integrated in Hadoop-Common-trunk #360 (See [http://hudson.zones.apache.org/hudson/job/Hadoop-Common-trunk/360/])
    HADOOP-6813. Add a new newInstance method in FileSystem that takes a user as argument


> Add a new newInstance method in FileSystem that takes a "user" as argument
> --------------------------------------------------------------------------
>
>                 Key: HADOOP-6813
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6813
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: fs
>            Reporter: Devaraj Das
>            Assignee: Devaraj Das
>             Fix For: 0.22.0
>
>         Attachments: h-6813.patch
>
>
> In order to implement HDFS-1000 for trunk, I need to have a newInstance API in FileSystem that takes a "user" as an argument.

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


[jira] Commented: (HADOOP-6813) Add a new newInstance method in FileSystem that takes a "user" as argument

Posted by "Devaraj Das (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HADOOP-6813?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12876490#action_12876490 ] 

Devaraj Das commented on HADOOP-6813:
-------------------------------------

bq. This issue went from created to committed in less than an hour. Changes to the FileSystem API should require more notice and explanation than was provided in this JIRA, though this looks like a reasonable convenience method to me. 

Hmm.. makes sense to wait longer for such changes. The main reason for having this API is to take care of libhdfs. There is an API in libhdfs that tries to connect on behalf of users. In order to support that API, I added the method in FileSystem. The other option would be to create an UGI object and pass it to newInstance but that would complicate things a bit (to do with creating UGI objects in C code, etc.).

> Add a new newInstance method in FileSystem that takes a "user" as argument
> --------------------------------------------------------------------------
>
>                 Key: HADOOP-6813
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6813
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: fs
>            Reporter: Devaraj Das
>            Assignee: Devaraj Das
>             Fix For: 0.22.0
>
>         Attachments: h-6813.patch
>
>
> In order to implement HDFS-1000 for trunk, I need to have a newInstance API in FileSystem that takes a "user" as an argument.

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


[jira] Updated: (HADOOP-6813) Add a new newInstance method in FileSystem that takes a "user" as argument

Posted by "Boris Shkolnik (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HADOOP-6813?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Boris Shkolnik updated HADOOP-6813:
-----------------------------------

          Status: Resolved  (was: Patch Available)
    Hadoop Flags: [Reviewed]
      Resolution: Fixed

+1
committed to trunk. Thanks Devaraj.

> Add a new newInstance method in FileSystem that takes a "user" as argument
> --------------------------------------------------------------------------
>
>                 Key: HADOOP-6813
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6813
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: fs
>            Reporter: Devaraj Das
>            Assignee: Devaraj Das
>             Fix For: 0.22.0
>
>         Attachments: h-6813.patch
>
>
> In order to implement HDFS-1000 for trunk, I need to have a newInstance API in FileSystem that takes a "user" as an argument.

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


[jira] Commented: (HADOOP-6813) Add a new newInstance method in FileSystem that takes a "user" as argument

Posted by "Hadoop QA (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HADOOP-6813?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12876463#action_12876463 ] 

Hadoop QA commented on HADOOP-6813:
-----------------------------------

+1 overall.  Here are the results of testing the latest attachment 
  http://issues.apache.org/jira/secure/attachment/12446533/h-6813.patch
  against trunk revision 952454.

    +1 @author.  The patch does not contain any @author tags.

    +1 tests included.  The patch appears to include 3 new or modified tests.

    +1 javadoc.  The javadoc tool did not generate any warning messages.

    +1 javac.  The applied patch does not increase the total number of javac compiler warnings.

    +1 findbugs.  The patch does not introduce any new Findbugs warnings.

    +1 release audit.  The applied patch does not increase the total number of release audit warnings.

    +1 core tests.  The patch passed core unit tests.

    +1 contrib tests.  The patch passed contrib unit tests.

Test results: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-h4.grid.sp2.yahoo.net/571/testReport/
Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-h4.grid.sp2.yahoo.net/571/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
Checkstyle results: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-h4.grid.sp2.yahoo.net/571/artifact/trunk/build/test/checkstyle-errors.html
Console output: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-h4.grid.sp2.yahoo.net/571/console

This message is automatically generated.

> Add a new newInstance method in FileSystem that takes a "user" as argument
> --------------------------------------------------------------------------
>
>                 Key: HADOOP-6813
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6813
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: fs
>            Reporter: Devaraj Das
>            Assignee: Devaraj Das
>             Fix For: 0.22.0
>
>         Attachments: h-6813.patch
>
>
> In order to implement HDFS-1000 for trunk, I need to have a newInstance API in FileSystem that takes a "user" as an argument.

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