hadoop-common-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Chris Douglas (JIRA)" <j...@apache.org>
Subject [jira] Updated: (HADOOP-4677) FileSystem.getFileBlockLocations() (aka default implementation for Local FileSystem) incorrect.
Date Tue, 25 Nov 2008 11:03:44 GMT

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

Chris Douglas updated HADOOP-4677:
----------------------------------

    Status: Open  (was: Patch Available)

A couple nits on the test case:
* This should use
{code}
new Path(System.getProperty("test.build.data", "/tmp"), "work-dir")
{code}
for its temporary space, rather than hard-coding {{build/test/data/work-dir}}.
* TestCase is a subclass of Assert, isn't it? Is there a particular reason to use the static
methods?

I also agree with Nicholas: if this is going to be corrected, it should handle its boundary
conditions coherently.

> FileSystem.getFileBlockLocations() (aka default implementation for Local FileSystem)
incorrect.
> -----------------------------------------------------------------------------------------------
>
>                 Key: HADOOP-4677
>                 URL: https://issues.apache.org/jira/browse/HADOOP-4677
>             Project: Hadoop Core
>          Issue Type: Bug
>          Components: fs
>    Affects Versions: 0.19.0, 0.18.2, 0.18.1, 0.19.1, 0.20.0
>            Reporter: Hong Tang
>             Fix For: 0.20.0
>
>         Attachments: hadoop-4677-for-trunk.patch, hadoop-4677-trunk.patch, hadoop-4677-trunk.patch,
hadoop-4677.patch
>
>
> The default implementation of FileSystem.getFileBlockLocations(FileStatus file, long
start, long len) seems to be wrong.

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