[GitHub] storm pull request #2583: STORM-2649 More detailed check of config serializa...

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

[GitHub] storm pull request #2583: STORM-2649 More detailed check of config serializa...

2018-04-02 Thread ghajos
Github user ghajos commented on a diff in the pull request: https://github.com/apache/storm/pull/2583#discussion_r178527233 --- Diff: storm-client/test/jvm/org/apache/storm/utils/UtilsTest.java --- @@ -173,4 +176,49 @@ public void

[GitHub] storm pull request #2583: STORM-2649 More detailed check of config serializa...

2018-03-06 Thread revans2
Github user revans2 commented on a diff in the pull request: https://github.com/apache/storm/pull/2583#discussion_r172667597 --- Diff: storm-client/test/jvm/org/apache/storm/utils/UtilsTest.java --- @@ -173,4 +176,49 @@ public void

[GitHub] storm pull request #2583: STORM-2649 More detailed check of config serializa...

2018-03-06 Thread revans2
Github user revans2 commented on a diff in the pull request: https://github.com/apache/storm/pull/2583#discussion_r172665583 --- Diff: storm-client/src/jvm/org/apache/storm/utils/Utils.java --- @@ -1030,8 +1034,35 @@ private static Object normalizeConfValue(Object obj) {

[GitHub] storm pull request #2583: STORM-2649 More detailed check of config serializa...

2018-03-06 Thread revans2
Github user revans2 commented on a diff in the pull request: https://github.com/apache/storm/pull/2583#discussion_r172666722 --- Diff: storm-client/src/jvm/org/apache/storm/utils/Utils.java --- @@ -1030,8 +1034,35 @@ private static Object normalizeConfValue(Object obj) {

[GitHub] storm pull request #2583: STORM-2649 More detailed check of config serializa...

2018-03-06 Thread revans2
Github user revans2 commented on a diff in the pull request: https://github.com/apache/storm/pull/2583#discussion_r172666072 --- Diff: storm-client/src/jvm/org/apache/storm/utils/Utils.java --- @@ -1030,8 +1034,35 @@ private static Object normalizeConfValue(Object obj) {

[GitHub] storm pull request #2583: STORM-2649 More detailed check of config serializa...

2018-03-05 Thread HeartSaVioR
Github user HeartSaVioR commented on a diff in the pull request: https://github.com/apache/storm/pull/2583#discussion_r172427524 --- Diff: storm-client/test/jvm/org/apache/storm/utils/UtilsTest.java --- @@ -18,16 +18,19 @@ package org.apache.storm.utils;

[GitHub] storm pull request #2583: STORM-2649 More detailed check of config serializa...

2018-03-03 Thread srdo
Github user srdo commented on a diff in the pull request: https://github.com/apache/storm/pull/2583#discussion_r172011662 --- Diff: storm-client/src/jvm/org/apache/storm/utils/Utils.java --- @@ -1030,8 +1034,34 @@ private static Object normalizeConfValue(Object obj) {

[GitHub] storm pull request #2583: STORM-2649 More detailed check of config serializa...

2018-03-03 Thread srdo
Github user srdo commented on a diff in the pull request: https://github.com/apache/storm/pull/2583#discussion_r172011663 --- Diff: storm-client/test/jvm/org/apache/storm/utils/UtilsTest.java --- @@ -173,4 +176,23 @@ public void

[GitHub] storm pull request #2583: STORM-2649 More detailed check of config serializa...

2018-03-03 Thread srdo
Github user srdo commented on a diff in the pull request: https://github.com/apache/storm/pull/2583#discussion_r172011690 --- Diff: storm-client/test/jvm/org/apache/storm/utils/UtilsTest.java --- @@ -173,4 +176,23 @@ public void

[GitHub] storm pull request #2583: STORM-2649 More detailed check of config serializa...

2018-03-02 Thread ghajos
GitHub user ghajos opened a pull request: https://github.com/apache/storm/pull/2583 STORM-2649 More detailed check of config serialization You can merge this pull request into a Git repository by running: $ git pull https://github.com/ghajos/storm STORM-2649 Alternatively