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 "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2016/02/29 09:09:18 UTC

[jira] [Commented] (HADOOP-11935) Provide optional native implementation of stat syscall.

    [ https://issues.apache.org/jira/browse/HADOOP-11935?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15171561#comment-15171561 ] 

ASF GitHub Bot commented on HADOOP-11935:
-----------------------------------------

GitHub user makefu opened a pull request:

    https://github.com/apache/hadoop/pull/81

    Shell.java: fix absolute path `/bin/ls`

    This commit uses `ls` from PATH instead of relying on `ls` being stored in `/bin/`. The only file according to the POSIX standard which must be stored in `/bin/` is `sh`.
    This fixes issues plaguing distributions like NixOS which dynamically stitch together a PATH.
    
    This fix is loosly related to https://issues.apache.org/jira/browse/HADOOP-11935 and more specifically related to https://mail-archives.apache.org/mod_mbox/hadoop-user/201512.mbox/%3CCAH2nEUgsSeoJTJkD4T8z=D18ECnRgQ3Qvz861gU5+NPSsNgT=A@mail.gmail.com%3E

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/makefu/hadoop patch-1

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/hadoop/pull/81.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #81
    
----
commit 4e8f93d8eecce551b1eebd531c71e3c7144c9210
Author: Felix Richter <gi...@syntax-fehler.de>
Date:   2016-02-29T08:08:06Z

    Shell.java: fix absolute path `/bin/ls`
    
    This commit uses `ls` from PATH instead of relying on `ls` being stored in `/bin/`. The only file according to the POSIX standard which must be stored in `/bin/` is `sh`.
    This fixes issues plaguing distributions like NixOS which dynamically stitch together a PATH.
    
    This fix is loosly related to https://issues.apache.org/jira/browse/HADOOP-11935 and more specifically related to https://mail-archives.apache.org/mod_mbox/hadoop-user/201512.mbox/%3CCAH2nEUgsSeoJTJkD4T8z=D18ECnRgQ3Qvz861gU5+NPSsNgT=A@mail.gmail.com%3E

----


> Provide optional native implementation of stat syscall.
> -------------------------------------------------------
>
>                 Key: HADOOP-11935
>                 URL: https://issues.apache.org/jira/browse/HADOOP-11935
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: fs, native
>            Reporter: Chris Nauroth
>         Attachments: HADOOP-11935-NativeIO-prelim.patch
>
>
> Currently, {{RawLocalFileSystem.DeprecatedRawLocalFileStatus#loadPermissionInfo}} is implemented as forking an {{ls}} command and parsing the output.  This was observed to be a bottleneck in YARN-3491.  This issue proposes an optional native implementation of a {{stat}} syscall through JNI.  We would maintain the existing code as a fallback for systems where the native code is not available.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)