[GitHub] storm pull request #2812: STORM-3203: add back in the permission updates

2018-09-04 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/2812 ---

[GitHub] storm pull request #2812: STORM-3203: add back in the permission updates

2018-08-27 Thread HeartSaVioR
Github user HeartSaVioR commented on a diff in the pull request: https://github.com/apache/storm/pull/2812#discussion_r213125537 --- Diff: storm-server/src/main/java/org/apache/storm/localizer/LocallyCachedTopologyBlob.java --- @@ -53,21 +54,24 @@ private final boolean

[GitHub] storm pull request #2812: STORM-3203: add back in the permission updates

2018-08-27 Thread revans2
Github user revans2 commented on a diff in the pull request: https://github.com/apache/storm/pull/2812#discussion_r213030742 --- Diff: storm-server/src/main/java/org/apache/storm/localizer/LocallyCachedTopologyBlob.java --- @@ -125,6 +129,12 @@ public long

[GitHub] storm pull request #2812: STORM-3203: add back in the permission updates

2018-08-27 Thread revans2
Github user revans2 commented on a diff in the pull request: https://github.com/apache/storm/pull/2812#discussion_r213030220 --- Diff: storm-server/src/main/java/org/apache/storm/localizer/LocallyCachedTopologyBlob.java --- @@ -53,21 +54,24 @@ private final boolean

[GitHub] storm pull request #2812: STORM-3203: add back in the permission updates

2018-08-26 Thread danny0405
Github user danny0405 commented on a diff in the pull request: https://github.com/apache/storm/pull/2812#discussion_r212851256 --- Diff: storm-server/src/main/java/org/apache/storm/localizer/LocallyCachedTopologyBlob.java --- @@ -125,6 +129,12 @@ public long

[GitHub] storm pull request #2812: STORM-3203: add back in the permission updates

2018-08-26 Thread HeartSaVioR
Github user HeartSaVioR commented on a diff in the pull request: https://github.com/apache/storm/pull/2812#discussion_r212838182 --- Diff: storm-server/src/main/java/org/apache/storm/localizer/LocallyCachedTopologyBlob.java --- @@ -53,21 +54,24 @@ private final boolean

[GitHub] storm pull request #2812: STORM-3203: add back in the permission updates

2018-08-26 Thread HeartSaVioR
Github user HeartSaVioR commented on a diff in the pull request: https://github.com/apache/storm/pull/2812#discussion_r212838119 --- Diff: storm-server/src/main/java/org/apache/storm/localizer/LocallyCachedTopologyBlob.java --- @@ -125,6 +129,12 @@ public long

[GitHub] storm pull request #2812: STORM-3203: add back in the permission updates

2018-08-24 Thread danny0405
Github user danny0405 commented on a diff in the pull request: https://github.com/apache/storm/pull/2812#discussion_r212787574 --- Diff: storm-server/src/main/java/org/apache/storm/localizer/LocallyCachedTopologyBlob.java --- @@ -230,6 +240,19 @@ public void commitNewVersion(long

[GitHub] storm pull request #2812: STORM-3203: add back in the permission updates

2018-08-24 Thread danny0405
Github user danny0405 commented on a diff in the pull request: https://github.com/apache/storm/pull/2812#discussion_r212787423 --- Diff: storm-server/src/main/java/org/apache/storm/localizer/LocallyCachedTopologyBlob.java --- @@ -125,6 +129,12 @@ public long

[GitHub] storm pull request #2812: STORM-3203: add back in the permission updates

2018-08-24 Thread danny0405
Github user danny0405 commented on a diff in the pull request: https://github.com/apache/storm/pull/2812#discussion_r212787378 --- Diff: storm-server/src/main/java/org/apache/storm/localizer/LocallyCachedTopologyBlob.java --- @@ -53,21 +54,24 @@ private final boolean

[GitHub] storm pull request #2812: STORM-3203: add back in the permission updates

2018-08-23 Thread revans2
GitHub user revans2 opened a pull request: https://github.com/apache/storm/pull/2812 STORM-3203: add back in the permission updates This adds back in some permission updates and a directory creation that was missed previously. I have manually tested it and it works.