[GitHub] activemq-artemis pull request #1783: ARTEMIS-1611 Added support for 1.x tran...

2018-02-05 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/activemq-artemis/pull/1783 ---

[GitHub] activemq-artemis pull request #1783: ARTEMIS-1611 Added support for 1.x tran...

2018-01-22 Thread mtaylor
Github user mtaylor commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1783#discussion_r163036074 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/server/transformer/ServerMessageImpl.java --- @@ -0,0 +1,190 @@ +/*

[GitHub] activemq-artemis pull request #1783: ARTEMIS-1611 Added support for 1.x tran...

2018-01-22 Thread mtaylor
Github user mtaylor commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1783#discussion_r163035962 --- Diff: artemis-commons/src/main/java/org/apache/activemq/artemis/utils/TypedProperties.java --- @@ -23,4 +23,7 @@ @Deprecated

[GitHub] activemq-artemis pull request #1783: ARTEMIS-1611 Added support for 1.x tran...

2018-01-22 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1783#discussion_r163011302 --- Diff: artemis-commons/src/main/java/org/apache/activemq/artemis/utils/TypedProperties.java --- @@ -23,4 +23,7 @@ @Deprecated

[GitHub] activemq-artemis pull request #1783: ARTEMIS-1611 Added support for 1.x tran...

2018-01-22 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1783#discussion_r163010905 --- Diff: artemis-commons/src/main/java/org/apache/activemq/artemis/utils/TypedProperties.java --- @@ -23,4 +23,7 @@ @Deprecated

[GitHub] activemq-artemis pull request #1783: ARTEMIS-1611 Added support for 1.x tran...

2018-01-18 Thread clebertsuconic
Github user clebertsuconic commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1783#discussion_r162391229 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/server/transformer/ServerMessageImpl.java --- @@ -0,0 +1,190 @@

[GitHub] activemq-artemis pull request #1783: ARTEMIS-1611 Added support for 1.x tran...

2018-01-18 Thread clebertsuconic
Github user clebertsuconic commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1783#discussion_r162389643 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/server/transformer/ServerMessageImpl.java --- @@ -0,0 +1,190 @@

[GitHub] activemq-artemis pull request #1783: ARTEMIS-1611 Added support for 1.x tran...

2018-01-18 Thread mtaylor
Github user mtaylor commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1783#discussion_r162356443 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/server/transformer/ServerMessageImpl.java --- @@ -0,0 +1,192 @@ +/*

[GitHub] activemq-artemis pull request #1783: ARTEMIS-1611 Added support for 1.x tran...

2018-01-17 Thread jmesnil
Github user jmesnil commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1783#discussion_r162103683 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/server/transformer/ServerMessageImpl.java --- @@ -0,0 +1,192 @@ +/*

[GitHub] activemq-artemis pull request #1783: ARTEMIS-1611 Added support for 1.x tran...

2018-01-17 Thread mtaylor
Github user mtaylor commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1783#discussion_r162051266 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/server/transformer/Transformer.java --- @@ -19,10 +19,18 @@ import

[GitHub] activemq-artemis pull request #1783: ARTEMIS-1611 Added support for 1.x tran...

2018-01-17 Thread mtaylor
Github user mtaylor commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1783#discussion_r162048935 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/server/transformer/Transformer.java --- @@ -19,10 +19,18 @@ import

[GitHub] activemq-artemis pull request #1783: ARTEMIS-1611 Added support for 1.x tran...

2018-01-17 Thread jmesnil
Github user jmesnil commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1783#discussion_r162028476 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/server/transformer/Transformer.java --- @@ -19,10 +19,18 @@ import

[GitHub] activemq-artemis pull request #1783: ARTEMIS-1611 Added support for 1.x tran...

2018-01-17 Thread mtaylor
Github user mtaylor commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1783#discussion_r162024692 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/server/transformer/Transformer.java --- @@ -19,10 +19,18 @@ import

[GitHub] activemq-artemis pull request #1783: ARTEMIS-1611 Added support for 1.x tran...

2018-01-17 Thread jmesnil
Github user jmesnil commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1783#discussion_r162015472 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/server/transformer/Transformer.java --- @@ -19,10 +19,18 @@ import

[GitHub] activemq-artemis pull request #1783: ARTEMIS-1611 Added support for 1.x tran...

2018-01-17 Thread mtaylor
GitHub user mtaylor opened a pull request: https://github.com/apache/activemq-artemis/pull/1783 ARTEMIS-1611 Added support for 1.x transformer API I've added support for the old API back. There was no real need to remove it other than some internal refactoring. I haven't written