[GitHub] storm pull request #2759: STORM-2947: Remove some deprecated methods from St...

2018-07-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/2759 ---

[GitHub] storm pull request #2759: STORM-2947: Remove some deprecated methods from St...

2018-07-12 Thread HeartSaVioR
Github user HeartSaVioR commented on a diff in the pull request: https://github.com/apache/storm/pull/2759#discussion_r202034158 --- Diff: storm-client/src/jvm/org/apache/storm/generated/ClusterSummary.java --- @@ -29,24 +29,21 @@ private static final

[GitHub] storm pull request #2759: STORM-2947: Remove some deprecated methods from St...

2018-07-12 Thread HeartSaVioR
Github user HeartSaVioR commented on a diff in the pull request: https://github.com/apache/storm/pull/2759#discussion_r202033999 --- Diff: storm-client/src/jvm/org/apache/storm/daemon/StormCommon.java --- @@ -85,11 +84,6 @@ public static StormCommon setInstance(StormCommon common)

[GitHub] storm pull request #2759: STORM-2947: Remove some deprecated methods from St...

2018-07-12 Thread srdo
Github user srdo commented on a diff in the pull request: https://github.com/apache/storm/pull/2759#discussion_r202028846 --- Diff: storm-client/src/jvm/org/apache/storm/generated/ClusterSummary.java --- @@ -29,24 +29,21 @@ private static final

[GitHub] storm pull request #2759: STORM-2947: Remove some deprecated methods from St...

2018-07-12 Thread srdo
Github user srdo commented on a diff in the pull request: https://github.com/apache/storm/pull/2759#discussion_r202000446 --- Diff: storm-client/src/jvm/org/apache/storm/generated/ClusterSummary.java --- @@ -29,24 +29,21 @@ private static final

[GitHub] storm pull request #2759: STORM-2947: Remove some deprecated methods from St...

2018-07-12 Thread srdo
Github user srdo commented on a diff in the pull request: https://github.com/apache/storm/pull/2759#discussion_r202000182 --- Diff: storm-client/src/jvm/org/apache/storm/daemon/StormCommon.java --- @@ -85,11 +84,6 @@ public static StormCommon setInstance(StormCommon common) {

[GitHub] storm pull request #2759: STORM-2947: Remove some deprecated methods from St...

2018-07-12 Thread srdo
Github user srdo commented on a diff in the pull request: https://github.com/apache/storm/pull/2759#discussion_r201999379 --- Diff: storm-client/src/jvm/org/apache/storm/daemon/StormCommon.java --- @@ -85,11 +84,6 @@ public static StormCommon setInstance(StormCommon common) {

[GitHub] storm pull request #2759: STORM-2947: Remove some deprecated methods from St...

2018-07-12 Thread srdo
Github user srdo commented on a diff in the pull request: https://github.com/apache/storm/pull/2759#discussion_r201999052 --- Diff: storm-client/src/jvm/org/apache/storm/utils/Utils.java --- @@ -781,7 +781,7 @@ private static SerializationDelegate getSerializationDelegate(Map

[GitHub] storm pull request #2759: STORM-2947: Remove some deprecated methods from St...

2018-07-12 Thread HeartSaVioR
Github user HeartSaVioR commented on a diff in the pull request: https://github.com/apache/storm/pull/2759#discussion_r201930522 --- Diff: storm-client/src/jvm/org/apache/storm/daemon/StormCommon.java --- @@ -85,11 +84,6 @@ public static StormCommon setInstance(StormCommon common)

[GitHub] storm pull request #2759: STORM-2947: Remove some deprecated methods from St...

2018-07-12 Thread HeartSaVioR
Github user HeartSaVioR commented on a diff in the pull request: https://github.com/apache/storm/pull/2759#discussion_r201931277 --- Diff: storm-client/src/jvm/org/apache/storm/generated/ClusterSummary.java --- @@ -29,24 +29,21 @@ private static final

[GitHub] storm pull request #2759: STORM-2947: Remove some deprecated methods from St...

2018-07-12 Thread HeartSaVioR
Github user HeartSaVioR commented on a diff in the pull request: https://github.com/apache/storm/pull/2759#discussion_r201930088 --- Diff: storm-client/src/jvm/org/apache/storm/utils/Utils.java --- @@ -781,7 +781,7 @@ private static SerializationDelegate

[GitHub] storm pull request #2759: STORM-2947: Remove some deprecated methods from St...

2018-07-10 Thread srdo
Github user srdo commented on a diff in the pull request: https://github.com/apache/storm/pull/2759#discussion_r201455774 --- Diff: storm-client/src/jvm/org/apache/storm/utils/Utils.java --- @@ -781,7 +781,7 @@ private static SerializationDelegate getSerializationDelegate(Map

[GitHub] storm pull request #2759: STORM-2947: Remove some deprecated methods from St...

2018-07-10 Thread srdo
GitHub user srdo opened a pull request: https://github.com/apache/storm/pull/2759 STORM-2947: Remove some deprecated methods from Storm 2.0.0. This handles some of the deprecated things mentioned in STORM-2947. There'll probably be followups to this PR to remove more functionality,