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

ASF GitHub Bot commented on TWILL-208:
--------------------------------------

Github user chtyim commented on a diff in the pull request:

    https://github.com/apache/twill/pull/26#discussion_r98118427
  
    --- Diff: 
twill-yarn/src/test/java/org/apache/twill/filesystem/FileContextLocationTest.java
 ---
    @@ -17,26 +17,36 @@
      */
     package org.apache.twill.filesystem;
     
    +import com.google.common.base.Throwables;
     import org.apache.hadoop.conf.Configuration;
    +import org.apache.hadoop.fs.Path;
    +import org.apache.hadoop.fs.permission.FsPermission;
     import org.apache.hadoop.hdfs.MiniDFSCluster;
    +import org.apache.hadoop.security.UserGroupInformation;
     import org.junit.AfterClass;
     import org.junit.BeforeClass;
     
     import java.io.IOException;
    +import java.security.PrivilegedAction;
     
     /**
      *
      */
     public class FileContextLocationTest extends LocationTestBase {
     
       public static MiniDFSCluster dfsCluster;
    +  private static UserGroupInformation testUGI;
     
       @BeforeClass
       public static void init() throws IOException {
         Configuration conf = new Configuration();
    -    conf.setBoolean("hdfs.permissions", true);
         conf.set(MiniDFSCluster.HDFS_MINIDFS_BASEDIR, 
tmpFolder.newFolder().getAbsolutePath());
         dfsCluster = new MiniDFSCluster.Builder(conf).numDataNodes(1).build();
    +    // make rooot world-writable so that we can create all location 
factories as unprivileged user
    --- End diff --
    
    root


> Location should have a way to set permissions when creating directories
> -----------------------------------------------------------------------
>
>                 Key: TWILL-208
>                 URL: https://issues.apache.org/jira/browse/TWILL-208
>             Project: Apache Twill
>          Issue Type: Bug
>            Reporter: Andreas Neumann
>            Assignee: Andreas Neumann
>             Fix For: 0.10.0
>
>
> That is, we need to introduce a method
> {code}
> boolean mkdirs(String permissions);
> {code}



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

Reply via email to