[GitHub] activemq-artemis pull request #1404: ARTEMIS-1294 Reverted TimedBuffer timeo...

2017-07-19 Thread franz1981
Github user franz1981 closed the pull request at: https://github.com/apache/activemq-artemis/pull/1404 --- 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

[GitHub] activemq-artemis issue #1410: ARTEMIS-1300 Deadlock when Core direct deliver...

2017-07-19 Thread clebertsuconic
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/1410 done ^^ amended... This PR is very safe to merge. no test regressions on the whole testsuite. --- If your project is set up for it, you can reply to this email and

[GitHub] activemq-artemis pull request #1410: ARTEMIS-1300 Deadlock when Core direct ...

2017-07-19 Thread clebertsuconic
Github user clebertsuconic commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1410#discussion_r128418363 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/server/Consumer.java --- @@ -19,9 +19,18 @@ import

[GitHub] activemq-artemis pull request #1410: ARTEMIS-1300 Deadlock when Core direct ...

2017-07-19 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1410#discussion_r128417008 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/server/Consumer.java --- @@ -19,9 +19,18 @@ import

[GitHub] activemq-artemis pull request #1410: ARTEMIS-1300 Deadlock when Core direct ...

2017-07-19 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1410#discussion_r128416905 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/QueueImpl.java --- @@ -828,6 +834,11 @@ public

[GitHub] activemq-artemis pull request #1410: ARTEMIS-1300 Deadlock when Core direct ...

2017-07-19 Thread clebertsuconic
GitHub user clebertsuconic opened a pull request: https://github.com/apache/activemq-artemis/pull/1410 ARTEMIS-1300 Deadlock when Core direct deliverying and AMQP receive You can merge this pull request into a Git repository by running: $ git pull

[ANNOUNCE] Call for Logo Result

2017-07-19 Thread Clebert Suconic
... and the winner is... https://blogs.apache.org/activemq/entry/new-activemq-logo Thanks for all the participants, both who voted and for all the submissions. -- Clebert Suconic

[GitHub] activemq pull request #244: AMQ-6689: Logo Maroon - Submission

2017-07-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/activemq/pull/244 --- 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 feature is

[GitHub] activemq pull request #242: AMQ-6689: Logo Colourful Modern - Submission

2017-07-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/activemq/pull/242 --- 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 feature is

[GitHub] activemq pull request #249: AMQ-6689 Logo Hex Colourful

2017-07-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/activemq/pull/249 --- 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 feature is

[GitHub] activemq pull request #253: Multiple ActiveMQ Logo Submission

2017-07-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/activemq/pull/253 --- 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 feature is

[GitHub] activemq pull request #251: Jason Brock multiple logo submissions

2017-07-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/activemq/pull/251 --- 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 feature is

[GitHub] activemq pull request #255: Added new ActiveMQ Logo files

2017-07-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/activemq/pull/255 --- 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 feature is

[GitHub] activemq pull request #248: AMQ-6689: Active Arrow A

2017-07-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/activemq/pull/248 --- 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 feature is

[GitHub] activemq pull request #245: AMQ-6689: New logo for ActiveMQ

2017-07-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/activemq/pull/245 --- 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 feature is

[GitHub] activemq pull request #243: AMQ-6689: Logo Maroon Lower Text - Submission

2017-07-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/activemq/pull/243 --- 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 feature is

[GitHub] activemq-artemis issue #1407: ARTEMIS-1276 fix JmsSendReceiveWithMessageExpi...

2017-07-19 Thread clebertsuconic
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/1407 >>> If it expired on client, how would it then get to DLQ queue on server? There's a special protocol on OpenWire to tell the server the message expired. On Artemis we

[GitHub] activemq-artemis issue #1407: ARTEMIS-1276 fix JmsSendReceiveWithMessageExpi...

2017-07-19 Thread jdanekrh
Github user jdanekrh commented on the issue: https://github.com/apache/activemq-artemis/pull/1407 > messages only expire on client on activemq5 while it's on server & client on artemis Did you mean expire on _server_ on activemq5? If it expired on client, how would it then

[GitHub] activemq pull request #255: Added new ActiveMQ Logo files

2017-07-19 Thread mtaylor
GitHub user mtaylor opened a pull request: https://github.com/apache/activemq/pull/255 Added new ActiveMQ Logo files You can merge this pull request into a Git repository by running: $ git pull https://github.com/mtaylor/activemq ActiveMQLogo Alternatively you can review and

Re: HEADS UP 1.5.6 and 2.2.0 release in 1 or 2 Weeks

2017-07-19 Thread Clebert Suconic
I keep pushing this back as I was doing some cleanup... everything is ready to go now... but I will cut the release on monday... rainy, stormy.. whatever.. monday is the day :) Sharp! So, don't make me break that promise by not breaking master till then.. although I can always revert a commit :)

[GitHub] activemq-artemis issue #1404: ARTEMIS-1294 Reverted TimedBuffer timeout poli...

2017-07-19 Thread clebertsuconic
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/1404 can you close this please? I cherry-picked manually --- 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

[GitHub] activemq-artemis issue #1407: ARTEMIS-1276 fix JmsSendReceiveWithMessageExpi...

2017-07-19 Thread clebertsuconic
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/1407 the test would take the consideration the messages only expire on client on activemq5 while it's on server & client on artemis. I had seen some other tests failing because of

[GitHub] activemq-artemis pull request #1409: ARTEMIS-1276 avoid System.exit in tests...

2017-07-19 Thread jdanekrh
Github user jdanekrh commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1409#discussion_r128231365 --- Diff: tests/activemq5-unit-tests/pom.xml --- @@ -460,6 +460,8 @@ **/org/apache/activemq/ProducerFlowControlTest.java

[GitHub] activemq-artemis issue #1409: ARTEMIS-1276 avoid System.exit in tests as Sur...

2017-07-19 Thread clebertsuconic
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/1409 Just remove the test then please ? --- 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