[GitHub] activemq-artemis pull request #1686: ARTEMIS-1534 added openwire logging (tr...

2017-12-04 Thread pgfox
GitHub user pgfox opened a pull request: https://github.com/apache/activemq-artemis/pull/1686 ARTEMIS-1534 added openwire logging (trace level) to log openwire commands that are received/sent by the broker. This is similar to the CORE protocol packet level logging in org/apac

Re: [VOTE] ActiveMQ Artemis becomes ActiveMQ 6

2017-12-04 Thread Francois Papon
+1 (non-binding) Francois Le 05/12/2017 à 00:32, Clebert Suconic a écrit : > Following on from the discussion, "[DISCUSS] Confusion surrounding the > ActiveMQ project roadmap" > > linked here for convenience : > - > http://activemq.2283324.n4.nabble.com/DISCUSS-Confusion-surrounding-the-ActiveM

Re: [VOTE] ActiveMQ Artemis becomes ActiveMQ 6

2017-12-04 Thread Michael André Pearce
+1 (non-binding) Cheers Mike > On 4 Dec 2017, at 20:32, Clebert Suconic wrote: > > Following on from the discussion, "[DISCUSS] Confusion surrounding the > ActiveMQ project roadmap" > > linked here for convenience : > - > http://activemq.2283324.n4.nabble.com/DISCUSS-Confusion-surrounding-the

[VOTE] ActiveMQ Artemis becomes ActiveMQ 6

2017-12-04 Thread Clebert Suconic
Following on from the discussion, "[DISCUSS] Confusion surrounding the ActiveMQ project roadmap" linked here for convenience : - http://activemq.2283324.n4.nabble.com/DISCUSS-Confusion-surrounding-the-ActiveMQ-project-roadmap-td4732935.html - http://activemq.2283324.n4.nabble.com/Re-DISCUSS-Conf

[GitHub] activemq-artemis issue #1679: ARTEMIS-1523 Allow MQTT with dynamic cluster

2017-12-04 Thread jbertram
Github user jbertram commented on the issue: https://github.com/apache/activemq-artemis/pull/1679 > For what I have analysed in the artemis code, the message-load-balancing-type is only set for queues configured in the broker.xml. I don't believe that's true. If you change yo

Re: Active MQ docker image

2017-12-04 Thread imranrazakhan
Hi, Any news for Official docker image for ActiveMQ? Regards, -- Sent from: http://activemq.2283324.n4.nabble.com/ActiveMQ-Dev-f2368404.html

[GitHub] activemq-artemis issue #1659: ARTEMIS-1516 - Ensure JNDI via Tomcat Resource...

2017-12-04 Thread michaelandrepearce
Github user michaelandrepearce commented on the issue: https://github.com/apache/activemq-artemis/pull/1659 Thats fine with me :) , i was just doing and updating you, that i just pushed the example and the docs i promised. ---

[GitHub] activemq-artemis issue #1659: ARTEMIS-1516 - Ensure JNDI via Tomcat Resource...

2017-12-04 Thread clebertsuconic
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/1659 I am working on some compatibility tests I will add into master.. and I will then merge this! ---

[GitHub] activemq-artemis pull request #1684: ARTEMIS-1498: Openwire internal headers...

2017-12-04 Thread clebertsuconic
Github user clebertsuconic commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1684#discussion_r154658085 --- Diff: artemis-protocols/artemis-amqp-protocol/src/main/java/org/apache/activemq/artemis/protocol/amqp/converter/CoreAmqpConverter.java ---

[GitHub] activemq-artemis issue #1659: ARTEMIS-1516 - Ensure JNDI via Tomcat Resource...

2017-12-04 Thread michaelandrepearce
Github user michaelandrepearce commented on the issue: https://github.com/apache/activemq-artemis/pull/1659 @clebertsuconic i have added a document page now, and also contributed in the example i had separately into the examples section. ---

[GitHub] activemq-artemis pull request #1684: ARTEMIS-1498: Openwire internal headers...

2017-12-04 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1684#discussion_r154642682 --- Diff: artemis-protocols/artemis-amqp-protocol/src/main/java/org/apache/activemq/artemis/protocol/amqp/converter/CoreAmqpConverter.java

[GitHub] activemq-artemis issue #1684: ARTEMIS-1498: Openwire internal headers should...

2017-12-04 Thread michaelandrepearce
Github user michaelandrepearce commented on the issue: https://github.com/apache/activemq-artemis/pull/1684 If this is specific to OpenWire headers, should this not be stripped during converting open wire messages within the open wire protocol support module, it shouldn't be leaking i

[GitHub] activemq-artemis issue #1679: ARTEMIS-1523 Allow MQTT with dynamic cluster

2017-12-04 Thread Skiler
Github user Skiler commented on the issue: https://github.com/apache/activemq-artemis/pull/1679 Hi @jbertram For what I have analysed in the artemis code, the message-load-balancing-type is only set for queues configured in the broker.xml. When you publish to an auto crea

Re: Contribute a properties-based SQL Provider

2017-12-04 Thread Martyn Taylor
Hi Jeff, I think I much prefer this approach of keeping the database specific logic (i.e. the SQL statements) separate from the implementation. This would allow us to have a pure JDBC API + SQL implementation, and adding support for a new vendor would just be a case of dropping in a new propertie

[GitHub] activemq-artemis pull request #1685: ARTEMIS-1533 Import subschema's so XMLL...

2017-12-04 Thread michaelandrepearce
GitHub user michaelandrepearce opened a pull request: https://github.com/apache/activemq-artemis/pull/1685 ARTEMIS-1533 Import subschema's so XMLLINT validates broker.xml You can merge this pull request into a Git repository by running: $ git pull https://github.com/michaeland

[GitHub] activemq-artemis pull request #1684: ARTEMIS-1498: Openwire internal headers...

2017-12-04 Thread jdanekrh
Github user jdanekrh commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1684#discussion_r154599253 --- Diff: artemis-protocols/artemis-amqp-protocol/src/main/java/org/apache/activemq/artemis/protocol/amqp/converter/CoreAmqpConverter.java --- @

[GitHub] activemq-artemis pull request #1684: ARTEMIS-1498: Openwire internal headers...

2017-12-04 Thread RaiSaurabh
GitHub user RaiSaurabh opened a pull request: https://github.com/apache/activemq-artemis/pull/1684 ARTEMIS-1498: Openwire internal headers should not be part of message… … properties You can merge this pull request into a Git repository by running: $ git pull https://github

[GitHub] activemq-artemis issue #1683: ARTEMIS-1532 Enable tests which are unintentio...

2017-12-04 Thread jdanekrh
Github user jdanekrh commented on the issue: https://github.com/apache/activemq-artemis/pull/1683 Well, I might've guessed the intent wrong in one or two cases, I guess. If I thought it is unintentional, I've either renamed the class (so that Surefire would pick it) or added t