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 "Eli Collins (JIRA)" <ji...@apache.org> on 2009/12/18 02:50:18 UTC

[jira] Created: (HADOOP-6455) Use /tmp for test root directory in FileContextTestHelper

Use /tmp for test root directory in FileContextTestHelper
---------------------------------------------------------

                 Key: HADOOP-6455
                 URL: https://issues.apache.org/jira/browse/HADOOP-6455
             Project: Hadoop Common
          Issue Type: Test
            Reporter: Eli Collins
            Assignee: Eli Collins


Per other tests set the test root directory to /tmp rather than /user.  Testing: 
- TestLocalFSFileContextMainOperations now passes when run from eclipse (rather than access non-existant directory).
- ant test-core still passes

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


[jira] Updated: (HADOOP-6455) Use /tmp for test root directory in FileContextTestHelper

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

Eli Collins updated HADOOP-6455:
--------------------------------

    Status: Patch Available  (was: Open)

> Use /tmp for test root directory in FileContextTestHelper
> ---------------------------------------------------------
>
>                 Key: HADOOP-6455
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6455
>             Project: Hadoop Common
>          Issue Type: Test
>            Reporter: Eli Collins
>            Assignee: Eli Collins
>         Attachments: hadoop-6455.patch, hadoop-6455.patch
>
>
> Per other tests set the test root directory to /tmp rather than /user.  Testing: 
> - TestLocalFSFileContextMainOperations now passes when run from eclipse (rather than access non-existant directory).
> - ant test-core still passes

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


[jira] Updated: (HADOOP-6455) Use /tmp for test root directory in FileContextTestHelper

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

Eli Collins updated HADOOP-6455:
--------------------------------

    Status: Open  (was: Patch Available)

> Use /tmp for test root directory in FileContextTestHelper
> ---------------------------------------------------------
>
>                 Key: HADOOP-6455
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6455
>             Project: Hadoop Common
>          Issue Type: Test
>            Reporter: Eli Collins
>            Assignee: Eli Collins
>         Attachments: hadoop-6455.patch, hadoop-6455.patch
>
>
> Per other tests set the test root directory to /tmp rather than /user.  Testing: 
> - TestLocalFSFileContextMainOperations now passes when run from eclipse (rather than access non-existant directory).
> - ant test-core still passes

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


[jira] Updated: (HADOOP-6455) Use /tmp for test root directory in FileContextTestHelper

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

Eli Collins updated HADOOP-6455:
--------------------------------

    Attachment: hadoop-6455.patch

This time w/o prefix.

> Use /tmp for test root directory in FileContextTestHelper
> ---------------------------------------------------------
>
>                 Key: HADOOP-6455
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6455
>             Project: Hadoop Common
>          Issue Type: Test
>            Reporter: Eli Collins
>            Assignee: Eli Collins
>         Attachments: hadoop-6455.patch, hadoop-6455.patch
>
>
> Per other tests set the test root directory to /tmp rather than /user.  Testing: 
> - TestLocalFSFileContextMainOperations now passes when run from eclipse (rather than access non-existant directory).
> - ant test-core still passes

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


[jira] Commented: (HADOOP-6455) Use /tmp for test root directory in FileContextTestHelper

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

Jakob Homan commented on HADOOP-6455:
-------------------------------------

Hmmm... make that HADOOP-5916

> Use /tmp for test root directory in FileContextTestHelper
> ---------------------------------------------------------
>
>                 Key: HADOOP-6455
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6455
>             Project: Hadoop Common
>          Issue Type: Test
>            Reporter: Eli Collins
>            Assignee: Eli Collins
>         Attachments: hadoop-6455.patch, hadoop-6455.patch
>
>
> Per other tests set the test root directory to /tmp rather than /user.  Testing: 
> - TestLocalFSFileContextMainOperations now passes when run from eclipse (rather than access non-existant directory).
> - ant test-core still passes

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


[jira] Updated: (HADOOP-6455) Use /tmp for test root directory in FileContextTestHelper

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

Eli Collins updated HADOOP-6455:
--------------------------------

    Attachment: hadoop-6455.patch

> Use /tmp for test root directory in FileContextTestHelper
> ---------------------------------------------------------
>
>                 Key: HADOOP-6455
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6455
>             Project: Hadoop Common
>          Issue Type: Test
>            Reporter: Eli Collins
>            Assignee: Eli Collins
>         Attachments: hadoop-6455.patch
>
>
> Per other tests set the test root directory to /tmp rather than /user.  Testing: 
> - TestLocalFSFileContextMainOperations now passes when run from eclipse (rather than access non-existant directory).
> - ant test-core still passes

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


[jira] Commented: (HADOOP-6455) Use /tmp for test root directory in FileContextTestHelper

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

Jakob Homan commented on HADOOP-6455:
-------------------------------------

There has been quite a bit of discussion about why /tmp is not a good idea (HADOOP:5916).  Is there any reason this code shouldn't standarize on buidl/test/data, as other tests have?

> Use /tmp for test root directory in FileContextTestHelper
> ---------------------------------------------------------
>
>                 Key: HADOOP-6455
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6455
>             Project: Hadoop Common
>          Issue Type: Test
>            Reporter: Eli Collins
>            Assignee: Eli Collins
>         Attachments: hadoop-6455.patch, hadoop-6455.patch
>
>
> Per other tests set the test root directory to /tmp rather than /user.  Testing: 
> - TestLocalFSFileContextMainOperations now passes when run from eclipse (rather than access non-existant directory).
> - ant test-core still passes

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


[jira] Updated: (HADOOP-6455) Use /tmp for test root directory in FileContextTestHelper

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

Eli Collins updated HADOOP-6455:
--------------------------------

    Resolution: Duplicate
        Status: Resolved  (was: Patch Available)

Thanks for the pointer Jakob, I'll close this as a dupe of HADOOP-5916 and comment there.

> Use /tmp for test root directory in FileContextTestHelper
> ---------------------------------------------------------
>
>                 Key: HADOOP-6455
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6455
>             Project: Hadoop Common
>          Issue Type: Test
>            Reporter: Eli Collins
>            Assignee: Eli Collins
>         Attachments: hadoop-6455.patch, hadoop-6455.patch
>
>
> Per other tests set the test root directory to /tmp rather than /user.  Testing: 
> - TestLocalFSFileContextMainOperations now passes when run from eclipse (rather than access non-existant directory).
> - ant test-core still passes

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


[jira] Commented: (HADOOP-6455) Use /tmp for test root directory in FileContextTestHelper

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

Hadoop QA commented on HADOOP-6455:
-----------------------------------

+1 overall.  Here are the results of testing the latest attachment 
  http://issues.apache.org/jira/secure/attachment/12428390/hadoop-6455.patch
  against trunk revision 892066.

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

    +1 tests included.  The patch appears to include 5 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/221/testReport/
Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-h4.grid.sp2.yahoo.net/221/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
Checkstyle results: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-h4.grid.sp2.yahoo.net/221/artifact/trunk/build/test/checkstyle-errors.html
Console output: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-h4.grid.sp2.yahoo.net/221/console

This message is automatically generated.

> Use /tmp for test root directory in FileContextTestHelper
> ---------------------------------------------------------
>
>                 Key: HADOOP-6455
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6455
>             Project: Hadoop Common
>          Issue Type: Test
>            Reporter: Eli Collins
>            Assignee: Eli Collins
>         Attachments: hadoop-6455.patch, hadoop-6455.patch
>
>
> Per other tests set the test root directory to /tmp rather than /user.  Testing: 
> - TestLocalFSFileContextMainOperations now passes when run from eclipse (rather than access non-existant directory).
> - ant test-core still passes

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


[jira] Commented: (HADOOP-6455) Use /tmp for test root directory in FileContextTestHelper

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

Hadoop QA commented on HADOOP-6455:
-----------------------------------

-1 overall.  Here are the results of testing the latest attachment 
  http://issues.apache.org/jira/secure/attachment/12428388/hadoop-6455.patch
  against trunk revision 892066.

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

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

    -1 patch.  The patch command could not apply the patch.

Console output: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-h1.grid.sp2.yahoo.net/32/console

This message is automatically generated.

> Use /tmp for test root directory in FileContextTestHelper
> ---------------------------------------------------------
>
>                 Key: HADOOP-6455
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6455
>             Project: Hadoop Common
>          Issue Type: Test
>            Reporter: Eli Collins
>            Assignee: Eli Collins
>         Attachments: hadoop-6455.patch
>
>
> Per other tests set the test root directory to /tmp rather than /user.  Testing: 
> - TestLocalFSFileContextMainOperations now passes when run from eclipse (rather than access non-existant directory).
> - ant test-core still passes

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


[jira] Updated: (HADOOP-6455) Use /tmp for test root directory in FileContextTestHelper

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

Eli Collins updated HADOOP-6455:
--------------------------------

    Status: Patch Available  (was: Open)

> Use /tmp for test root directory in FileContextTestHelper
> ---------------------------------------------------------
>
>                 Key: HADOOP-6455
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6455
>             Project: Hadoop Common
>          Issue Type: Test
>            Reporter: Eli Collins
>            Assignee: Eli Collins
>         Attachments: hadoop-6455.patch
>
>
> Per other tests set the test root directory to /tmp rather than /user.  Testing: 
> - TestLocalFSFileContextMainOperations now passes when run from eclipse (rather than access non-existant directory).
> - ant test-core still passes

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