[ https://issues.apache.org/jira/browse/TWILL-208?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15836912#comment-15836912 ]
ASF GitHub Bot commented on TWILL-208: -------------------------------------- Github user poornachandra commented on a diff in the pull request: https://github.com/apache/twill/pull/26#discussion_r97682463 --- Diff: twill-common/src/main/java/org/apache/twill/filesystem/LocalLocation.java --- @@ -98,8 +98,8 @@ public OutputStream getOutputStream() throws IOException { @Override public OutputStream getOutputStream(String permission) throws IOException { - ensureDirectory(file.getParentFile()); Set<PosixFilePermission> permissions = parsePermissions(permission); + ensureDirectory(file.getParentFile(), permissions); --- End diff -- Do we want to use the file permissions for the directory too? > 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)