hadoop-common-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Patrick Kling (JIRA)" <j...@apache.org>
Subject [jira] Updated: (HADOOP-7060) A more elegant FileSystem#listCorruptFileBlocks API
Date Thu, 09 Dec 2010 04:06:01 GMT

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

Patrick Kling updated HADOOP-7060:
----------------------------------

    Attachment: HADOOP-7060.patch

ant test passes.

ant test-patch results:

{code}
     [exec] -1 overall.  
     [exec] 
     [exec]     +1 @author.  The patch does not contain any @author tags.
     [exec] 
     [exec]     -1 tests included.  The patch doesn't appear to include any new or modified
tests.
     [exec]                         Please justify why no new tests are needed for this patch.
     [exec]                         Also please list what manual steps were performed to verify
this patch.
     [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 (version 1.3.9)
warnings.
     [exec] 
     [exec]     +1 release audit.  The applied patch does not increase the total number of
release audit warnings.
     [exec] 
     [exec]     +1 system test framework.  The patch passed system test framework compile.
{code}

A test for this change is included in HDFS-1533 (TestListCorruptFileBlocks).

> A more elegant FileSystem#listCorruptFileBlocks API
> ---------------------------------------------------
>
>                 Key: HADOOP-7060
>                 URL: https://issues.apache.org/jira/browse/HADOOP-7060
>             Project: Hadoop Common
>          Issue Type: Bug
>            Reporter: Hairong Kuang
>            Assignee: Patrick Kling
>         Attachments: HADOOP-7060.patch
>
>
> I'd like to change the newly added listCorruptFileBlocks signature to be:
> {code}
> /**
> * Get all files with corrupt blocks under the given path
> */
> RemoteIterator<Path> listCorruptFileBlocks(Path src) throws IOException;
> {code}
> This new API does not expose "cookie" to user although underlying implementation may
still need to invoke multiple RPCs to get the whole list.

-- 
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