[GitHub] franz1981 commented on issue #2523: ARTEMIS-1977 ASYNCIO can reduce sys-calls to retrieve I/O events

2019-01-25 Thread GitBox
franz1981 commented on issue #2523: ARTEMIS-1977 ASYNCIO can reduce sys-calls to retrieve I/O events URL: https://github.com/apache/activemq-artemis/pull/2523#issuecomment-457728883 About the memory barriers? Are the standard implementation used on the c++11 memory model and I have asked

[GitHub] franz1981 edited a comment on issue #2523: ARTEMIS-1977 ASYNCIO can reduce sys-calls to retrieve I/O events

2019-01-25 Thread GitBox
franz1981 edited a comment on issue #2523: ARTEMIS-1977 ASYNCIO can reduce sys-calls to retrieve I/O events URL: https://github.com/apache/activemq-artemis/pull/2523#issuecomment-457728883 @clebertsuconic About the memory barriers? Are the standard implementation used on the c++11 memory

[GitHub] clebertsuconic commented on a change in pull request #2523: ARTEMIS-1977 ASYNCIO can reduce sys-calls to retrieve I/O events

2019-01-25 Thread GitBox
clebertsuconic commented on a change in pull request #2523: ARTEMIS-1977 ASYNCIO can reduce sys-calls to retrieve I/O events URL: https://github.com/apache/activemq-artemis/pull/2523#discussion_r251130524 ## File path:

[GitHub] franz1981 edited a comment on issue #2523: ARTEMIS-1977 ASYNCIO can reduce sys-calls to retrieve I/O events

2019-01-25 Thread GitBox
franz1981 edited a comment on issue #2523: ARTEMIS-1977 ASYNCIO can reduce sys-calls to retrieve I/O events URL: https://github.com/apache/activemq-artemis/pull/2523#issuecomment-457728883 @clebertsuconic About the memory barriers? Are the standard implementation used on the c++11 memory

[GitHub] clebertsuconic commented on a change in pull request #2529: Refactor the native library build code

2019-01-30 Thread GitBox
clebertsuconic commented on a change in pull request #2529: Refactor the native library build code URL: https://github.com/apache/activemq-artemis/pull/2529#discussion_r252335209 ## File path: artemis-native/README.md ## @@ -0,0 +1,87 @@ +# Introduction + + +This is a

[GitHub] franz1981 commented on issue #2479: ARTEMIS-2211 Refactor ByteBuffer pooling, alignment and zeroing

2019-01-30 Thread GitBox
franz1981 commented on issue #2479: ARTEMIS-2211 Refactor ByteBuffer pooling, alignment and zeroing URL: https://github.com/apache/activemq-artemis/pull/2479#issuecomment-459005431 @michaelandrepearce @clebertsuconic Yep, the CI has run on it and it is ok :+1: I will use it right

[GitHub] franz1981 edited a comment on issue #2479: ARTEMIS-2211 Refactor ByteBuffer pooling, alignment and zeroing

2019-01-30 Thread GitBox
franz1981 edited a comment on issue #2479: ARTEMIS-2211 Refactor ByteBuffer pooling, alignment and zeroing URL: https://github.com/apache/activemq-artemis/pull/2479#issuecomment-459005431 @michaelandrepearce @clebertsuconic Yep, the CI has run on it and it is ok :+1: I will use it

[GitHub] clebertsuconic opened a new pull request #2531: ARTEMIS-2205 Fixing Divert Example

2019-01-30 Thread GitBox
clebertsuconic opened a new pull request #2531: ARTEMIS-2205 Fixing Divert Example URL: https://github.com/apache/activemq-artemis/pull/2531 This is simply fixing the example under examples/features/standard/divert Other tests are passing. No additional tests are needed as the

[GitHub] orpiske commented on a change in pull request #2529: Refactor the native library build code

2019-01-30 Thread GitBox
orpiske commented on a change in pull request #2529: Refactor the native library build code URL: https://github.com/apache/activemq-artemis/pull/2529#discussion_r252332040 ## File path: artemis-native/README.md ## @@ -0,0 +1,87 @@ +# Introduction + + +This is a simple

[GitHub] orpiske commented on a change in pull request #2529: Refactor the native library build code

2019-01-30 Thread GitBox
orpiske commented on a change in pull request #2529: Refactor the native library build code URL: https://github.com/apache/activemq-artemis/pull/2529#discussion_r252338396 ## File path: artemis-native/README.md ## @@ -0,0 +1,87 @@ +# Introduction + + +This is a simple

[GitHub] michaelandrepearce commented on a change in pull request #2529: Refactor the native library build code

2019-01-30 Thread GitBox
michaelandrepearce commented on a change in pull request #2529: Refactor the native library build code URL: https://github.com/apache/activemq-artemis/pull/2529#discussion_r252339116 ## File path: artemis-native/README.md ## @@ -0,0 +1,87 @@ +# Introduction + + +This is a

[GitHub] clebertsuconic commented on a change in pull request #2529: Refactor the native library build code

2019-01-30 Thread GitBox
clebertsuconic commented on a change in pull request #2529: Refactor the native library build code URL: https://github.com/apache/activemq-artemis/pull/2529#discussion_r252323631 ## File path: artemis-native/README.md ## @@ -0,0 +1,87 @@ +# Introduction + + +This is a

[GitHub] franz1981 edited a comment on issue #2479: ARTEMIS-2211 Refactor ByteBuffer pooling, alignment and zeroing

2019-01-30 Thread GitBox
franz1981 edited a comment on issue #2479: ARTEMIS-2211 Refactor ByteBuffer pooling, alignment and zeroing URL: https://github.com/apache/activemq-artemis/pull/2479#issuecomment-459005431 @michaelandrepearce @clebertsuconic Yep, the CI has run on it and it is ok :+1: I will use it

[GitHub] michaelandrepearce commented on a change in pull request #2529: Refactor the native library build code

2019-01-30 Thread GitBox
michaelandrepearce commented on a change in pull request #2529: Refactor the native library build code URL: https://github.com/apache/activemq-artemis/pull/2529#discussion_r252344727 ## File path: artemis-native/README.md ## @@ -0,0 +1,87 @@ +# Introduction + + +This is a

[GitHub] orpiske commented on a change in pull request #2529: Refactor the native library build code

2019-01-30 Thread GitBox
orpiske commented on a change in pull request #2529: Refactor the native library build code URL: https://github.com/apache/activemq-artemis/pull/2529#discussion_r252329904 ## File path: artemis-native/README.md ## @@ -0,0 +1,87 @@ +# Introduction + + +This is a simple

[GitHub] clebertsuconic commented on a change in pull request #2529: Refactor the native library build code

2019-01-30 Thread GitBox
clebertsuconic commented on a change in pull request #2529: Refactor the native library build code URL: https://github.com/apache/activemq-artemis/pull/2529#discussion_r252339673 ## File path: artemis-native/README.md ## @@ -0,0 +1,87 @@ +# Introduction + + +This is a

[GitHub] asfgit closed pull request #2531: ARTEMIS-2205 Fixing Divert Example

2019-01-30 Thread GitBox
asfgit closed pull request #2531: ARTEMIS-2205 Fixing Divert Example URL: https://github.com/apache/activemq-artemis/pull/2531 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] jbertram opened a new pull request #2532: ARTEMIS-2243 user/role ops for PropertiesLoginModule via mgmnt

2019-01-30 Thread GitBox
jbertram opened a new pull request #2532: ARTEMIS-2243 user/role ops for PropertiesLoginModule via mgmnt URL: https://github.com/apache/activemq-artemis/pull/2532 This is an automated message from the Apache Git Service. To

[GitHub] asfgit merged pull request #2535: NO-JIRA update docs for gitbox

2019-01-31 Thread GitBox
asfgit merged pull request #2535: NO-JIRA update docs for gitbox URL: https://github.com/apache/activemq-artemis/pull/2535 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252893793 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252892610 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252892610 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252893830 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252898402 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252900294 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252900294 ## File path:

[GitHub] onlyMIT commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
onlyMIT commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252910278 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252912679 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252912679 ## File path:

[GitHub] onlyMIT commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
onlyMIT commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252915920 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252916808 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252918453 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252918453 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252896834 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252899919 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252899919 ## File path:

[GitHub] onlyMIT commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
onlyMIT commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252915302 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252917956 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252891076 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252892610 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252898402 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252900294 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252900294 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252912679 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252915242 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252916780 ## File path:

[GitHub] onlyMIT commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
onlyMIT commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252918661 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252918453 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252896834 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252891076 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252898402 ## File path:

[GitHub] onlyMIT commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
onlyMIT commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252914838 ## File path:

[GitHub] michaelandrepearce commented on issue #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
michaelandrepearce commented on issue #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#issuecomment-459583628 @onlyMIT to help with addressing the comments i sent a pr to your branch with a number of

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252892706 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252892610 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252897572 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252900294 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252900294 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252892610 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252892610 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252892610 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252896834 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252897572 ## File path:

[GitHub] onlyMIT commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
onlyMIT commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252914292 ## File path:

[GitHub] onlyMIT commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
onlyMIT commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252914292 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252917296 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252917296 ## File path:

[GitHub] onlyMIT commented on issue #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
onlyMIT commented on issue #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#issuecomment-459587945 @michaelandrepearce thanks to your review, I will address the comments.

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252920340 ## File path:

[GitHub] gaohoward commented on issue #2534: ARTEMIS-1058 Jars in web tmp dir locked on Windows

2019-01-31 Thread GitBox
gaohoward commented on issue #2534: ARTEMIS-1058 Jars in web tmp dir locked on Windows URL: https://github.com/apache/activemq-artemis/pull/2534#issuecomment-459631440 @clebertsuconic I'm not aware of the "encoded" issue, where can I get more information about it? The first commit is

[GitHub] onlyMIT commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
onlyMIT commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252919168 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252920340 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252920340 ## File path:

[GitHub] gaohoward commented on issue #2534: ARTEMIS-1058 Jars in web tmp dir locked on Windows

2019-01-31 Thread GitBox
gaohoward commented on issue #2534: ARTEMIS-1058 Jars in web tmp dir locked on Windows URL: https://github.com/apache/activemq-artemis/pull/2534#issuecomment-459588365 @clebertsuconic The commit ae00423 was valid. However on Windows when running artemis as a service, stopping the

[GitHub] gaohoward edited a comment on issue #2534: ARTEMIS-1058 Jars in web tmp dir locked on Windows

2019-01-31 Thread GitBox
gaohoward edited a comment on issue #2534: ARTEMIS-1058 Jars in web tmp dir locked on Windows URL: https://github.com/apache/activemq-artemis/pull/2534#issuecomment-459588365 @clebertsuconic The commit ae00423 was valid. However on Windows when running artemis as a service, stopping the

[GitHub] clebertsuconic commented on issue #2534: ARTEMIS-1058 Jars in web tmp dir locked on Windows

2019-01-31 Thread GitBox
clebertsuconic commented on issue #2534: ARTEMIS-1058 Jars in web tmp dir locked on Windows URL: https://github.com/apache/activemq-artemis/pull/2534#issuecomment-459602236 I believe there is an issue with the original commit with encoded strings. I believe you should use a file and some

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-02-01 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252963097 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-02-01 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252965032 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-02-01 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252965032 ## File path:

[GitHub] onlyMIT commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
onlyMIT commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252921461 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252920340 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-01-31 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252920340 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-02-01 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r252963097 ## File path:

[GitHub] onlyMIT commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-02-01 Thread GitBox
onlyMIT commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r253012354 ## File path:

[GitHub] onlyMIT commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-02-01 Thread GitBox
onlyMIT commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r253014812 ## File path:

[GitHub] onlyMIT commented on issue #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-02-01 Thread GitBox
onlyMIT commented on issue #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#issuecomment-459702347 @michaelandrepearce This PR has updated. `SESSIONS` is still in MQTTSession, do not move to

[GitHub] michaelandrepearce commented on issue #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-02-01 Thread GitBox
michaelandrepearce commented on issue #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#issuecomment-459709207 There is the factory if thats the case where a map could be passed into each handler during

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-02-01 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r253105287 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-02-01 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r253105898 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-02-01 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r253117361 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-02-03 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r253318615 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-02-03 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r253318691 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-02-03 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r253318615 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-02-03 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r253318691 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-02-03 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r253318691 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-02-03 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r253318691 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-02-03 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r253318957 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-02-03 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r253318957 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-02-03 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r253318957 ## File path:

[GitHub] michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu…

2019-02-03 Thread GitBox
michaelandrepearce commented on a change in pull request #2528: ARTEMIS-2226 last consumer connection should close the previous consu… URL: https://github.com/apache/activemq-artemis/pull/2528#discussion_r253319407 ## File path:

<    6   7   8   9   10   11   12   13   >