[GitHub] activemq-artemis issue #736: Remove thread.sleep in JDBCTest

2016-08-30 Thread jbertram
Github user jbertram commented on the issue: https://github.com/apache/activemq-artemis/pull/736 Is this related to a particular JIRA? --- 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

[GitHub] activemq-artemis pull request #738: ARTEMIS-696 Broker fails when client sen...

2016-08-30 Thread gaohoward
Github user gaohoward commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/738#discussion_r76818244 --- Diff: artemis-protocols/artemis-proton-plug/src/main/java/org/proton/plug/context/server/ProtonServerReceiverContext.java --- @@ -116,7

[GitHub] activemq-artemis pull request #740: ARTEMIS-706 No Keep Alives from Broker

2016-08-30 Thread gaohoward
Github user gaohoward commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/740#discussion_r76818458 --- Diff: artemis-protocols/artemis-openwire-protocol/src/main/java/org/apache/activemq/artemis/core/protocol/openwire/OpenWireConnection.java ---

[GitHub] activemq-artemis issue #738: ARTEMIS-696 Broker fails when client sends mess...

2016-08-30 Thread clebertsuconic
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/738 @gaohoward I started a run on my box.. np --- 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

[GitHub] activemq-artemis issue #738: ARTEMIS-696 Broker fails when client sends mess...

2016-08-30 Thread gaohoward
Github user gaohoward commented on the issue: https://github.com/apache/activemq-artemis/pull/738 @clebertsuconic no I didn't run the whole suite. --- 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

[GitHub] activemq-artemis pull request #740: ARTEMIS-706 No Keep Alives from Broker

2016-08-30 Thread clebertsuconic
Github user clebertsuconic commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/740#discussion_r76802109 --- Diff: artemis-protocols/artemis-openwire-protocol/src/main/java/org/apache/activemq/artemis/core/protocol/openwire/OpenWireConnection.java

ApacheCon Seville CFP closes September 9th

2016-08-30 Thread Rich Bowen
It's traditional. We wait for the last minute to get our talk proposals in for conferences. Well, the last minute has arrived. The CFP for ApacheCon Seville closes on September 9th, which is less than 2 weeks away. It's time to get your talks in, so that we can make this the best ApacheCon yet.

Re: Contributing to the Apache Docker Account

2016-08-30 Thread Clebert Suconic
The way I always wanted this was to have the CLI doing it: ./artemis -docker-create (parameters...) But I always told that this doesn't make sense in the docker world. On Mon, Aug 29, 2016 at 7:16 PM, Victor wrote: > Dear devs, > > I'm afraid I cannot create a good

[GitHub] activemq-artemis pull request #740: ARTEMIS-706 No Keep Alives from Broker

2016-08-30 Thread clebertsuconic
Github user clebertsuconic commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/740#discussion_r76800400 --- Diff: artemis-protocols/artemis-openwire-protocol/src/main/java/org/apache/activemq/artemis/core/protocol/openwire/OpenWireConnection.java

[GitHub] activemq-artemis pull request #740: ARTEMIS-706 No Keep Alives from Broker

2016-08-30 Thread clebertsuconic
Github user clebertsuconic commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/740#discussion_r76800800 --- Diff: artemis-protocols/artemis-openwire-protocol/src/main/java/org/apache/activemq/artemis/core/protocol/openwire/OpenWireConnection.java

[GitHub] activemq-artemis pull request #740: ARTEMIS-706 No Keep Alives from Broker

2016-08-30 Thread clebertsuconic
Github user clebertsuconic commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/740#discussion_r76801952 --- Diff: artemis-protocols/artemis-openwire-protocol/src/main/java/org/apache/activemq/artemis/core/protocol/openwire/OpenWireConnection.java

[GitHub] activemq-artemis issue #738: ARTEMIS-696 Broker fails when client sends mess...

2016-08-30 Thread clebertsuconic
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/738 @mtaylor Yep.. I always do :) --- 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

[GitHub] activemq-artemis pull request #738: ARTEMIS-696 Broker fails when client sen...

2016-08-30 Thread clebertsuconic
Github user clebertsuconic commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/738#discussion_r76802715 --- Diff: artemis-protocols/artemis-proton-plug/src/main/java/org/proton/plug/context/server/ProtonServerReceiverContext.java --- @@ -116,7

Re: Contributing to the Apache Docker Account

2016-08-30 Thread John D. Ament
Hi Victor, Right, what I was thinking is that both the readme and the dockerfile live in a subdirectory. This way chroot works correctly. John On Tue, Aug 30, 2016 at 1:50 AM Victor wrote: > Hi John, > > No we can't, dockerhub will chroot to the directory where the

Re: Contributing to the Apache Docker Account

2016-08-30 Thread John D. Ament
You're correct, it doesn't. Docker is all about preconfigured images. You could use this to generate a dockerfile, based on some base image that we ship. John On Tue, Aug 30, 2016 at 11:09 AM Clebert Suconic wrote: > The way I always wanted this was to have the CLI

[GitHub] activemq-artemis issue #738: ARTEMIS-696 Broker fails when client sends mess...

2016-08-30 Thread clebertsuconic
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/738 This is apparently good. I need to do some tests today. Did you run the whole testsuite on it? --- If your project is set up for it, you can reply to this email and have

[GitHub] activemq-artemis pull request #741: ARTEMIS-702 JMS 2 spec violation for rec...

2016-08-30 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/741 ARTEMIS-702 JMS 2 spec violation for receiveBody You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis

[GitHub] activemq-artemis pull request #742: ARTEMIS-657: ensure management response ...

2016-08-30 Thread grs
GitHub user grs opened a pull request: https://github.com/apache/activemq-artemis/pull/742 ARTEMIS-657: ensure management response bodies are encoded into amqp You can merge this pull request into a Git repository by running: $ git pull https://github.com/grs/activemq-artemis

[GitHub] activemq-artemis pull request #743: ARTEMIS-699 group ID reaper tx bug

2016-08-30 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/activemq-artemis/pull/743 --- 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 pull request #742: ARTEMIS-657: ensure management response ...

2016-08-30 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/activemq-artemis/pull/742 --- 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 pull request #738: ARTEMIS-696 Broker fails when client sen...

2016-08-30 Thread clebertsuconic
Github user clebertsuconic commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/738#discussion_r76881991 --- Diff: artemis-protocols/artemis-proton-plug/src/main/java/org/proton/plug/context/server/ProtonServerReceiverContext.java --- @@ -116,7

Re: Contributing to the Apache Docker Account

2016-08-30 Thread Victor
Hi John, No, that is not possible as the rest of the sources are not available and therefor the build cannot be done, i.e: cannot: ADD .. in a dockerfile 2016-08-30 7:27 GMT-07:00 John D. Ament : > Hi Victor, > > Right, what I was thinking is that both the readme and the

[GitHub] activemq-artemis pull request #743: ARTEMIS-699 group ID reaper tx bug

2016-08-30 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/743 ARTEMIS-699 group ID reaper tx bug You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis ARTEMIS-699

[GitHub] activemq-artemis pull request #738: ARTEMIS-696 Broker fails when client sen...

2016-08-30 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/activemq-artemis/pull/738 --- 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 #740: ARTEMIS-706 No Keep Alives from Broker

2016-08-30 Thread gaohoward
Github user gaohoward commented on the issue: https://github.com/apache/activemq-artemis/pull/740 @clebertsuconic I just updated my branch --- 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

[GitHub] activemq-artemis issue #741: ARTEMIS-702 JMS 2 spec violation for receiveBod...

2016-08-30 Thread clebertsuconic
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/741 I'm running a few tests. --- 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

[GitHub] activemq-artemis pull request #740: ARTEMIS-706 No Keep Alives from Broker

2016-08-30 Thread gaohoward
Github user gaohoward commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/740#discussion_r76913251 --- Diff: artemis-protocols/artemis-openwire-protocol/src/main/java/org/apache/activemq/artemis/core/protocol/openwire/OpenWireConnection.java ---

[GitHub] activemq-artemis pull request #740: ARTEMIS-706 No Keep Alives from Broker

2016-08-30 Thread gaohoward
Github user gaohoward commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/740#discussion_r76913484 --- Diff: artemis-protocols/artemis-openwire-protocol/src/main/java/org/apache/activemq/artemis/core/protocol/openwire/OpenWireConnection.java ---

Re: Contributing to the Apache Docker Account

2016-08-30 Thread John D. Ament
Hi Victor, Ok, I think that's based on some outdated information. There's still some discussions on going on other mailing lists, but it looks like right now any concept of a snapshot docker container shouldn't be used. Meaning what you have right now will be perfect. John On Tue, Aug 30,

Re: Contributing to the Apache Docker Account

2016-08-30 Thread Victor
Hi John, > Ok, I think that's based on some outdated information. You might be thinking of docker itself where it is perfectly possible. In docker hub, the issue 292 is still open. > There's still some discussions on going on other mailing