[jira] [Commented] (MAPREDUCE-7159) FrameworkUploader: ensure proper permissions of generated framework tar.gz if restrictive umask is used

2018-12-06 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-7159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16712033#comment-16712033 ] Hudson commented on MAPREDUCE-7159: --- SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit

[jira] [Commented] (MAPREDUCE-7159) FrameworkUploader: ensure proper permissions of generated framework tar.gz if restrictive umask is used

2018-12-06 Thread Jason Lowe (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-7159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16712010#comment-16712010 ] Jason Lowe commented on MAPREDUCE-7159: --- Thanks for updating the patch! +1 lgtm. Committing

[jira] [Commented] (MAPREDUCE-7159) FrameworkUploader: ensure proper permissions of generated framework tar.gz if restrictive umask is used

2018-12-06 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-7159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16711844#comment-16711844 ] Hadoop QA commented on MAPREDUCE-7159: -- | (/) *{color:green}+1 overall{color}* | \\ \\ || Vote

[jira] [Commented] (MAPREDUCE-7159) FrameworkUploader: ensure proper permissions of generated framework tar.gz if restrictive umask is used

2018-12-06 Thread Peter Bacsko (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-7159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16711776#comment-16711776 ] Peter Bacsko commented on MAPREDUCE-7159: - Thanks for the comments [~jlowe], attached patch

[jira] [Commented] (MAPREDUCE-7159) FrameworkUploader: ensure proper permissions of generated framework tar.gz if restrictive umask is used

2018-12-06 Thread Jason Lowe (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-7159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16711728#comment-16711728 ] Jason Lowe commented on MAPREDUCE-7159: --- Looks good overall, just a few nits. The program will

[jira] [Commented] (MAPREDUCE-7159) FrameworkUploader: ensure proper permissions of generated framework tar.gz if restrictive umask is used

2018-12-06 Thread Peter Bacsko (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-7159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16711194#comment-16711194 ] Peter Bacsko commented on MAPREDUCE-7159: - Thanks [~wilfreds] for reviewing. [~jlowe] could

[jira] [Commented] (MAPREDUCE-7159) FrameworkUploader: ensure proper permissions of generated framework tar.gz if restrictive umask is used

2018-12-05 Thread Wilfred Spiegelenburg (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-7159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16710874#comment-16710874 ] Wilfred Spiegelenburg commented on MAPREDUCE-7159: -- Patch looks good to me +1 (non

[jira] [Commented] (MAPREDUCE-7159) FrameworkUploader: ensure proper permissions of generated framework tar.gz if restrictive umask is used

2018-12-05 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-7159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16710116#comment-16710116 ] Hadoop QA commented on MAPREDUCE-7159: -- | (/) *{color:green}+1 overall{color}* | \\ \\ || Vote

[jira] [Commented] (MAPREDUCE-7159) FrameworkUploader: ensure proper permissions of generated framework tar.gz if restrictive umask is used

2018-12-05 Thread Peter Bacsko (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-7159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16710089#comment-16710089 ] Peter Bacsko commented on MAPREDUCE-7159: - [~wilfreds] I updated the patch. I walk the path

[jira] [Commented] (MAPREDUCE-7159) FrameworkUploader: ensure proper permissions of generated framework tar.gz if restrictive umask is used

2018-12-04 Thread Wilfred Spiegelenburg (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-7159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16709538#comment-16709538 ] Wilfred Spiegelenburg commented on MAPREDUCE-7159: -- Thank you for the updated patch

[jira] [Commented] (MAPREDUCE-7159) FrameworkUploader: ensure proper permissions of generated framework tar.gz if restrictive umask is used

2018-12-04 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-7159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16708733#comment-16708733 ] Hadoop QA commented on MAPREDUCE-7159: -- | (/) *{color:green}+1 overall{color}* | \\ \\ || Vote

[jira] [Commented] (MAPREDUCE-7159) FrameworkUploader: ensure proper permissions of generated framework tar.gz if restrictive umask is used

2018-12-04 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-7159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16708734#comment-16708734 ] Hadoop QA commented on MAPREDUCE-7159: -- | (/) *{color:green}+1 overall{color}* | \\ \\ || Vote

[jira] [Commented] (MAPREDUCE-7159) FrameworkUploader: ensure proper permissions of generated framework tar.gz if restrictive umask is used

2018-12-04 Thread Peter Bacsko (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-7159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16708675#comment-16708675 ] Peter Bacsko commented on MAPREDUCE-7159: - Patch v5: I slightly refactored the handling of

[jira] [Commented] (MAPREDUCE-7159) FrameworkUploader: ensure proper permissions of generated framework tar.gz if restrictive umask is used

2018-12-04 Thread Peter Bacsko (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-7159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16708653#comment-16708653 ] Peter Bacsko commented on MAPREDUCE-7159: - [~wilfreds] 1. This seems to be reasonable. 2.

[jira] [Commented] (MAPREDUCE-7159) FrameworkUploader: ensure proper permissions of generated framework tar.gz if restrictive umask is used

2018-12-03 Thread Wilfred Spiegelenburg (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-7159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16707882#comment-16707882 ] Wilfred Spiegelenburg commented on MAPREDUCE-7159: -- Thank you [~pbacsko] I have two

[jira] [Commented] (MAPREDUCE-7159) FrameworkUploader: ensure proper permissions of generated framework tar.gz if restrictive umask is used

2018-11-26 Thread Peter Bacsko (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-7159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16698672#comment-16698672 ] Peter Bacsko commented on MAPREDUCE-7159: - [~wilfreds] could you take a quick look at this?

[jira] [Commented] (MAPREDUCE-7159) FrameworkUploader: ensure proper permissions of generated framework tar.gz if restrictive umask is used

2018-11-23 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-7159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16697535#comment-16697535 ] Hadoop QA commented on MAPREDUCE-7159: -- | (/) *{color:green}+1 overall{color}* | \\ \\ || Vote

[jira] [Commented] (MAPREDUCE-7159) FrameworkUploader: ensure proper permissions of generated framework tar.gz if restrictive umask is used

2018-11-23 Thread Peter Bacsko (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-7159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16697510#comment-16697510 ] Peter Bacsko commented on MAPREDUCE-7159: - Fixing testcase + checkstyle problem. >

[jira] [Commented] (MAPREDUCE-7159) FrameworkUploader: ensure proper permissions of generated framework tar.gz if restrictive umask is used

2018-11-23 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/MAPREDUCE-7159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16697359#comment-16697359 ] Hadoop QA commented on MAPREDUCE-7159: -- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote ||