[GitHub] activemq-artemis pull request #2136: Revert "[ARTEMIS-1819] Missing fields o...

2018-06-11 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/2136 Revert "[ARTEMIS-1819] Missing fields on listAllConsumersAsJSON, list… …ConsumersAsJSON and listConnectionsAsJSON" This reverts commit c3fbd1b9e479f87898e0acd53f386d588c99763

Abount "KahaDB Cleanup" of durable subscribers including pending messages

2018-06-11 Thread t-watana
My understanding is that, KahaDB rewrite the KAHA_SUBSCRIPTION_COMMAND to latest journal file for ease of deletion the oldeset jorunal log, when the durable subscriber has no pending message. But, in the following case, "Durable Subscriber C and D" aren't rewritten to latest journal despite they

Some useless commands when the durable subscriber is destroyed (Offline Timeout)

2018-06-11 Thread t-watana
Hi all, I use ActiveMQ 5.13.1 for MQTT. It seems that ActiveMQ writes some useless commands, when the durable subscriber is destroyed by Offline Timeout. For example, the following situation: - Topic a, b, c - Durable Subscriber A, B, C - "Durable Subscriber A" subscribe ONLY "Topic a" - "Durable

[GitHub] activemq-artemis pull request #2134: ARTEMIS-1922 clarify doc on updating xi...

2018-06-11 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/activemq-artemis/pull/2134 ---

[GitHub] activemq-artemis pull request #2135: ARTEMIS-1924 Add amqpIdleTimeout

2018-06-11 Thread clebertsuconic
Github user clebertsuconic commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2135#discussion_r194569617 --- Diff: tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/amqp/AmqpNoHearbeatsTest.java --- @@ -0,0 +1

[GitHub] activemq-artemis pull request #2135: ARTEMIS-1924 Add amqpIdleTimeout

2018-06-11 Thread clebertsuconic
Github user clebertsuconic commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2135#discussion_r194569421 --- Diff: tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/amqp/AmqpNoHearbeatsTest.java --- @@ -0,0 +1

[GitHub] activemq-artemis pull request #2135: ARTEMIS-1924 Add amqpIdleTimeout

2018-06-11 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/activemq-artemis/pull/2135 ---

[GitHub] activemq-artemis pull request #2135: ARTEMIS-1924 Add amqpIdleTimeout

2018-06-11 Thread tabish121
Github user tabish121 commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2135#discussion_r194568008 --- Diff: tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/amqp/AmqpNoHearbeatsTest.java --- @@ -0,0 +1,83 @

[GitHub] activemq-artemis issue #2135: ARTEMIS-1924 Add amqpIdleTimeout

2018-06-11 Thread jbertram
Github user jbertram commented on the issue: https://github.com/apache/activemq-artemis/pull/2135 Code looks good. Add a paragraph to the docs and it should be good to go. ---

[GitHub] activemq-artemis pull request #2135: ARTEMIS-1924 Add amqpIdleTimeout

2018-06-11 Thread clebertsuconic
GitHub user clebertsuconic opened a pull request: https://github.com/apache/activemq-artemis/pull/2135 ARTEMIS-1924 Add amqpIdleTimeout You can merge this pull request into a Git repository by running: $ git pull https://github.com/clebertsuconic/activemq-artemis ARTEMIS-1924

[GitHub] activemq-artemis pull request #2134: ARTEMIS-1922 clarify doc on updating xi...

2018-06-11 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/2134 ARTEMIS-1922 clarify doc on updating xincluded files You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis ARTEM

[GitHub] activemq-artemis issue #2128: ARTEMIS-1916 Remove Jmx ArtemisRMIServerSocket...

2018-06-11 Thread andytaylor
Github user andytaylor commented on the issue: https://github.com/apache/activemq-artemis/pull/2128 I would create a single PR. Also Im not sure this is a complete fix. There is still code in there that sets up an SSL socket layer. Also I think we need to understand

[GitHub] activemq-artemis issue #2133: ARTEMIS-1923 fix constructing binding journali...

2018-06-11 Thread jbertram
Github user jbertram commented on the issue: https://github.com/apache/activemq-artemis/pull/2133 A test here would be useful. Even something as simple as adding some getters and verifying the properties are set as expected would work, I think. ---

the useless commands are written into KahaDB when the durable subscriber is destroyed by "offline timeout"

2018-06-11 Thread t-watana
Hi all, I use ActiveMQ 5.13.1 for MQTT. It seems that ActivMQ writes some useless commands, when the durable subscriber is destroyed by Offline Timeout. For example, the following situation: - Topic a, b, c - Durable Subscriber A, B, C - "Durable Subscriber A" subscribe ONLY "Topic a" - "Durable

[GitHub] activemq-artemis issue #2132: ARTEMIS-1893: Treat the detach of a link with ...

2018-06-11 Thread k-wall
Github user k-wall commented on the issue: https://github.com/apache/activemq-artemis/pull/2132 That should be the checkstyle violation resolved. ---

[GitHub] activemq-artemis pull request #2133: ARTEMIS-1923 fix constructing binding j...

2018-06-11 Thread wy96f
GitHub user wy96f opened a pull request: https://github.com/apache/activemq-artemis/pull/2133 ARTEMIS-1923 fix constructing binding journalimpl bug You can merge this pull request into a Git repository by running: $ git pull https://github.com/wy96f/activemq-artemis binding_jo

[GitHub] activemq-artemis issue #2132: ARTEMIS-1893: Treat the detach of a link with ...

2018-06-11 Thread andytaylor
Github user andytaylor commented on the issue: https://github.com/apache/activemq-artemis/pull/2132 actually on second glance I think what you have done is correct ---

[GitHub] activemq-artemis issue #2132: ARTEMIS-1893: Treat the detach of a link with ...

2018-06-11 Thread andytaylor
Github user andytaylor commented on the issue: https://github.com/apache/activemq-artemis/pull/2132 you do have 1 checkstyle violation that needs fixing ---

[GitHub] activemq-artemis issue #2132: ARTEMIS-1893: Treat the detach of a link with ...

2018-06-11 Thread k-wall
Github user k-wall commented on the issue: https://github.com/apache/activemq-artemis/pull/2132 Thanks for the feedback. I did initially consider making the change in `ProtonServerSenderContext#close(boolean)` but I couldn't see a way organise for the `closed` field of the detach per

[GitHub] activemq-artemis issue #2132: ARTEMIS-1893: Treat the detach of a link with ...

2018-06-11 Thread andytaylor
Github user andytaylor commented on the issue: https://github.com/apache/activemq-artemis/pull/2132 There is code in ProtonServerSenderContext and ProtonServerReceiverContextthat handles how to treat this scenario, maybe it sthis that needs updating rather than handling this in 2 plac

[GitHub] activemq-artemis pull request #2132: ARTEMIS-1893: Treat the detach of a lin...

2018-06-11 Thread k-wall
GitHub user k-wall opened a pull request: https://github.com/apache/activemq-artemis/pull/2132 ARTEMIS-1893: Treat the detach of a link with expiry policy of link-detatch as if closed flag were true. I believe this patch will resolve the issue I reported. You can merge this pull re