[GitHub] activemq-artemis pull request #677: ARTEMIS-595 additional JSON fixes

2016-07-28 Thread clebertsuconic
Github user clebertsuconic commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/677#discussion_r72635740 --- Diff: artemis-core-client/src/main/java/org/apache/activemq/artemis/api/core/JsonUtil.java --- @@ -295,6 +295,13 @@ else if (jsonValue =

[GitHub] activemq-artemis pull request #677: ARTEMIS-595 additional JSON fixes

2016-07-28 Thread jbertram
Github user jbertram commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/677#discussion_r72633668 --- Diff: artemis-core-client/src/main/java/org/apache/activemq/artemis/api/core/JsonUtil.java --- @@ -295,6 +295,13 @@ else if (jsonValue == Json

[GitHub] activemq-artemis pull request #677: ARTEMIS-595 additional JSON fixes

2016-07-28 Thread clebertsuconic
Github user clebertsuconic commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/677#discussion_r72632899 --- Diff: tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/management/CoreMessagingProxy.java --- @@ -30,

[GitHub] activemq-artemis pull request #677: ARTEMIS-595 additional JSON fixes

2016-07-28 Thread clebertsuconic
Github user clebertsuconic commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/677#discussion_r72631140 --- Diff: artemis-core-client/src/main/java/org/apache/activemq/artemis/api/core/JsonUtil.java --- @@ -295,6 +295,13 @@ else if (jsonValue =

[GitHub] activemq-artemis pull request #677: ARTEMIS-595 additional JSON fixes

2016-07-28 Thread clebertsuconic
Github user clebertsuconic commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/677#discussion_r72631347 --- Diff: tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/management/ActiveMQServerControlTest.java ---

[GitHub] activemq-artemis pull request #677: ARTEMIS-595 additional JSON fixes

2016-07-28 Thread clebertsuconic
Github user clebertsuconic commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/677#discussion_r72631247 --- Diff: tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/management/ActiveMQServerControlTest.java ---

[GitHub] activemq-artemis pull request #677: ARTEMIS-595 additional JSON fixes

2016-07-28 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/677 ARTEMIS-595 additional JSON fixes You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis JSONfixes Alternatively