[jira] [Resolved] (TWILL-208) Location should have a way to set permissions when creating directories

2017-01-26 Thread Andreas Neumann (JIRA)
[ https://issues.apache.org/jira/browse/TWILL-208?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Andreas Neumann resolved TWILL-208. --- Resolution: Fixed > Location should have a way to set permissions when creating directories >

[jira] [Commented] (TWILL-208) Location should have a way to set permissions when creating directories

2017-01-26 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TWILL-208?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15840748#comment-15840748 ] ASF GitHub Bot commented on TWILL-208: -- Github user asfgit closed the pull request at:

[GitHub] twill issue #26: (TWILL-208) add Location.mkdirs(String permissions)

2017-01-26 Thread poornachandra
Github user poornachandra commented on the issue: https://github.com/apache/twill/pull/26 LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[jira] [Commented] (TWILL-208) Location should have a way to set permissions when creating directories

2017-01-26 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TWILL-208?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15840710#comment-15840710 ] ASF GitHub Bot commented on TWILL-208: -- Github user poornachandra commented on the issue:

[jira] [Commented] (TWILL-208) Location should have a way to set permissions when creating directories

2017-01-26 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TWILL-208?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15840683#comment-15840683 ] ASF GitHub Bot commented on TWILL-208: -- Github user chtyim commented on a diff in the pull request:

[GitHub] twill pull request #26: (TWILL-208) add Location.mkdirs(String permissions)

2017-01-26 Thread chtyim
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

[jira] [Commented] (TWILL-181) Control the maximum number of retries for failed application starts

2017-01-26 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TWILL-181?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15840427#comment-15840427 ] ASF GitHub Bot commented on TWILL-181: -- Github user poornachandra commented on a diff in the pull

[GitHub] twill pull request #23: (TWILL-181) allow setting the maximum number of retr...

2017-01-26 Thread poornachandra
Github user poornachandra commented on a diff in the pull request: https://github.com/apache/twill/pull/23#discussion_r98088342 --- Diff: twill-yarn/src/main/java/org/apache/twill/internal/appmaster/RunningContainers.java --- @@ -113,9 +117,11 @@ public Integer apply(BitSet

[GitHub] twill pull request #23: (TWILL-181) allow setting the maximum number of retr...

2017-01-26 Thread serranom
Github user serranom commented on a diff in the pull request: https://github.com/apache/twill/pull/23#discussion_r98083899 --- Diff: twill-yarn/src/main/java/org/apache/twill/internal/appmaster/RunningContainers.java --- @@ -113,9 +117,11 @@ public Integer apply(BitSet input) {

[jira] [Commented] (TWILL-208) Location should have a way to set permissions when creating directories

2017-01-26 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TWILL-208?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15840321#comment-15840321 ] ASF GitHub Bot commented on TWILL-208: -- Github user anew commented on the issue:

[GitHub] twill issue #26: (TWILL-208) add Location.mkdirs(String permissions)

2017-01-26 Thread anew
Github user anew commented on the issue: https://github.com/apache/twill/pull/26 Squashing commits now. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so,

[jira] [Commented] (TWILL-208) Location should have a way to set permissions when creating directories

2017-01-26 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TWILL-208?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15840320#comment-15840320 ] ASF GitHub Bot commented on TWILL-208: -- Github user poornachandra commented on the issue:

[GitHub] twill issue #26: (TWILL-208) add Location.mkdirs(String permissions)

2017-01-26 Thread poornachandra
Github user poornachandra commented on the issue: https://github.com/apache/twill/pull/26 LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GitHub] twill issue #26: (TWILL-208) add Location.mkdirs(String permissions)

2017-01-26 Thread poornachandra
Github user poornachandra commented on the issue: https://github.com/apache/twill/pull/26 Just one comment on javadoc, please fix and merge 👍 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[jira] [Commented] (TWILL-208) Location should have a way to set permissions when creating directories

2017-01-26 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TWILL-208?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15840317#comment-15840317 ] ASF GitHub Bot commented on TWILL-208: -- Github user poornachandra commented on the issue:

[jira] [Commented] (TWILL-181) Control the maximum number of retries for failed application starts

2017-01-26 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TWILL-181?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15840301#comment-15840301 ] ASF GitHub Bot commented on TWILL-181: -- Github user poornachandra commented on a diff in the pull

[jira] [Commented] (TWILL-208) Location should have a way to set permissions when creating directories

2017-01-26 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TWILL-208?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15840260#comment-15840260 ] ASF GitHub Bot commented on TWILL-208: -- Github user poornachandra commented on a diff in the pull

[GitHub] twill pull request #26: (TWILL-208) add Location.mkdirs(String permissions)

2017-01-26 Thread poornachandra
Github user poornachandra commented on a diff in the pull request: https://github.com/apache/twill/pull/26#discussion_r98070266 --- Diff: twill-common/src/main/java/org/apache/twill/filesystem/LocalLocation.java --- @@ -251,6 +251,34 @@ public boolean mkdirs() throws IOException {

[jira] [Commented] (TWILL-208) Location should have a way to set permissions when creating directories

2017-01-26 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TWILL-208?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15840171#comment-15840171 ] ASF GitHub Bot commented on TWILL-208: -- Github user gokulavasan commented on the issue:

[GitHub] twill issue #26: (TWILL-208) add Location.mkdirs(String permissions)

2017-01-26 Thread gokulavasan
Github user gokulavasan commented on the issue: https://github.com/apache/twill/pull/26 LGTM 👍 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if

[GitHub] twill issue #23: (TWILL-181) allow setting the maximum number of retries per...

2017-01-26 Thread serranom
Github user serranom commented on the issue: https://github.com/apache/twill/pull/23 @hsaputra, since I have a bug to fix, the updated request will trigger the rebuild. thanks. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[jira] [Commented] (TWILL-181) Control the maximum number of retries for failed application starts

2017-01-26 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TWILL-181?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15839677#comment-15839677 ] ASF GitHub Bot commented on TWILL-181: -- Github user serranom commented on a diff in the pull request:

[GitHub] twill pull request #23: (TWILL-181) allow setting the maximum number of retr...

2017-01-26 Thread poornachandra
Github user poornachandra commented on a diff in the pull request: https://github.com/apache/twill/pull/23#discussion_r97944230 --- Diff: twill-yarn/src/main/java/org/apache/twill/internal/appmaster/RunningContainers.java --- @@ -113,9 +117,11 @@ public Integer apply(BitSet

[jira] [Commented] (TWILL-181) Control the maximum number of retries for failed application starts

2017-01-26 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/TWILL-181?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15839417#comment-15839417 ] ASF GitHub Bot commented on TWILL-181: -- Github user poornachandra commented on a diff in the pull