[GitHub] activemq-artemis pull request #1472: ARTEMIS-1353 Initial replication of lar...

2017-08-18 Thread clebertsuconic
GitHub user clebertsuconic opened a pull request: https://github.com/apache/activemq-artemis/pull/1472 ARTEMIS-1353 Initial replication of large messages out of executor This is based on the work @jbertram made at the github pr #1466 and the discussions we had there You can merge t

[GitHub] activemq-artemis issue #1466: ARTEMIS-1353 ensure replication packet order

2017-08-18 Thread jbertram
Github user jbertram commented on the issue: https://github.com/apache/activemq-artemis/pull/1466 Clebert worked up a simpler solution so I'm closing this. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project d

[GitHub] activemq-artemis pull request #1466: ARTEMIS-1353 ensure replication packet ...

2017-08-18 Thread jbertram
Github user jbertram closed the pull request at: https://github.com/apache/activemq-artemis/pull/1466 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the fe

[GitHub] activemq-artemis issue #1466: ARTEMIS-1353 ensure replication packet order

2017-08-18 Thread clebertsuconic
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/1466 I'm not sure how you would do that. I would send 10 and some logic that would resume it. Like new runnable () {resumeCall(); } This seemed a simple

[GitHub] activemq-artemis pull request #1471: ARTEMIS-1350 - Update management consol...

2017-08-18 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/activemq-artemis/pull/1471 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feat

[GitHub] activemq-artemis pull request #1471: ARTEMIS-1350 - Update management consol...

2017-08-18 Thread mtaylor
Github user mtaylor commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1471#discussion_r133951178 --- Diff: artemis-hawtio/artemis-plugin/src/main/webapp/plugin/html/consumers.html --- @@ -44,9 +44,30 @@

[GitHub] activemq-artemis pull request #1471: ARTEMIS-1350 - Update management consol...

2017-08-18 Thread mtaylor
Github user mtaylor commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1471#discussion_r133948157 --- Diff: artemis-hawtio/artemis-plugin/src/main/webapp/plugin/html/consumers.html --- @@ -44,9 +44,30 @@

[GitHub] activemq-artemis pull request #1471: ARTEMIS-1350 - Update management consol...

2017-08-18 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1471#discussion_r133925691 --- Diff: artemis-hawtio/artemis-plugin/src/main/webapp/plugin/html/consumers.html --- @@ -44,9 +44,30 @@

[GitHub] activemq-artemis pull request #1471: ARTEMIS-1350 - Update management consol...

2017-08-18 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1471#discussion_r133925554 --- Diff: artemis-core-client/src/main/java/org/apache/activemq/artemis/api/core/management/ActiveMQServerControl.java --- @@ -773,6 +

[GitHub] activemq-artemis issue #1466: ARTEMIS-1353 ensure replication packet order

2017-08-18 Thread mtaylor
Github user mtaylor commented on the issue: https://github.com/apache/activemq-artemis/pull/1466 @clebertsuconic Rather than having all this code to synchronise the sending of large files via the same thread and executor. Why not just use an executor with a blocking queue for all?

[GitHub] activemq-artemis pull request #1471: ARTEMIS-1350 - Update management consol...

2017-08-18 Thread andytaylor
GitHub user andytaylor opened a pull request: https://github.com/apache/activemq-artemis/pull/1471 ARTEMIS-1350 - Update management console to expose client side objects https://issues.apache.org/jira/browse/ARTEMIS-1350 You can merge this pull request into a Git repository by runni