[GitHub] activemq-artemis issue #1655: ARTEMIS-1514 Large message fix

2017-11-13 Thread clebertsuconic
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/1655 I'm still refactoring large messages, but this change is independent as it may be picked on previous versions (1.x for instance). ---

[GitHub] activemq-artemis pull request #1655: ARTEMIS-1514 Large message fix

2017-11-13 Thread clebertsuconic
GitHub user clebertsuconic opened a pull request: https://github.com/apache/activemq-artemis/pull/1655 ARTEMIS-1514 Large message fix I'm doing an overal improvement on large message support for AMQP However this commit is just about a Bug on the converter. It will be

[GitHub] activemq-artemis pull request #1653: ARTEMIS-1510 refactor Maven poms

2017-11-13 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/activemq-artemis/pull/1653 ---

[GitHub] activemq-artemis issue #1654: More StompTest coverage and Stomp fix

2017-11-13 Thread clebertsuconic
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/1654 Kudos on a well done job! ---

[GitHub] activemq-artemis pull request #1654: More StompTest coverage and Stomp fix

2017-11-13 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/activemq-artemis/pull/1654 ---

[GitHub] activemq pull request #264: AMQ-6861 Allow customisation of network bridge c...

2017-11-13 Thread splatch
GitHub user splatch opened a pull request: https://github.com/apache/activemq/pull/264 AMQ-6861 Allow customisation of network bridge creation logic. You can merge this pull request into a Git repository by running: $ git pull https://github.com/splatch/activemq AMQ-6861

[GitHub] activemq-artemis pull request #1653: ARTEMIS-1510 refactor Maven poms

2017-11-13 Thread jbertram
Github user jbertram commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1653#discussion_r150556595 --- Diff: artemis-core-client/pom.xml --- @@ -65,6 +71,12 @@ test + org.hamcrest --- End

[GitHub] activemq-artemis pull request #1653: ARTEMIS-1510 refactor Maven poms

2017-11-13 Thread clebertsuconic
Github user clebertsuconic commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1653#discussion_r150548048 --- Diff: artemis-core-client/pom.xml --- @@ -65,6 +71,12 @@ test + org.hamcrest ---

[GitHub] activemq-artemis pull request #1654: More StompTest coverage and Stomp fix

2017-11-13 Thread mtaylor
GitHub user mtaylor opened a pull request: https://github.com/apache/activemq-artemis/pull/1654 More StompTest coverage and Stomp fix You can merge this pull request into a Git repository by running: $ git pull https://github.com/mtaylor/activemq-artemis StompTests

[GitHub] activemq-artemis issue #1647: ARTEMIS-1365 Advisory consumers listed in Cons...

2017-11-13 Thread gaohoward
Github user gaohoward commented on the issue: https://github.com/apache/activemq-artemis/pull/1647 I'll update this again. (Adding some doc about the newly added properties) ---

[GitHub] activemq-artemis issue #1628: ARTEMIS-1364: Enable internal sorting in Hawti...

2017-11-13 Thread RaiSaurabh
Github user RaiSaurabh commented on the issue: https://github.com/apache/activemq-artemis/pull/1628 Will fix and commit the indentation issue. ---

[GitHub] activemq-artemis issue #1628: ARTEMIS-1364: Enable internal sorting in Hawti...

2017-11-13 Thread RaiSaurabh
Github user RaiSaurabh commented on the issue: https://github.com/apache/activemq-artemis/pull/1628 Added the code to support sorting on the server side. Request @mtaylor to please review. ---