[GitHub] activemq-artemis issue #1466: ARTEMIS-1353 ensure replication packet order

2017-08-16 Thread clebertsuconic
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/1466 I want to do some review on the extra wait. Wait for me to merge this ? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] activemq-artemis issue #1466: ARTEMIS-1353 ensure replication packet order

2017-08-16 Thread jbertram
Github user jbertram commented on the issue: https://github.com/apache/activemq-artemis/pull/1466 The wait is in ReplicationManager#sendStartSyncMessage. It's essentially the same kind of thing that's done in ReplicationManager#sendSynchronizationDone. BTW, I ran the full

How to use ActiveMQCPP work on AIX??

2017-08-16 Thread yupub
I build ActiveMQCPP on AIX with compiler xlC, But it not work well. On Linux is ok. It suspend on transport->start. Who use ActiveMQCPP on aix? -- View this message in context: http://activemq.2283324.n4.nabble.com/How-to-use-ActiveMQCPP-work-on-AIX-tp4729715.html Sent from the ActiveMQ - Dev

[GitHub] activemq-artemis issue #1466: ARTEMIS-1353 ensure replication packet order

2017-08-16 Thread clebertsuconic
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/1466 Where did you add the wait ? --- 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 issue #1466: ARTEMIS-1353 ensure replication packet order

2017-08-16 Thread clebertsuconic
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/1466 Please wait me to merge this? I want to review this carefully tomorrow. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] activemq-artemis pull request #1466: ARTEMIS-1353 ensure replication packet ...

2017-08-16 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/1466 ARTEMIS-1353 ensure replication packet order Incorrect ordering of replication packets may happen because of useExecutor parameter in the sendReplicatePacket method.

[question] Gluster file system

2017-08-16 Thread Clebert Suconic
Has anyone tried activemq 5 with shared storage and gluster file system? If so what challenges did you have? I'm wondering what I would need to change in artemis to support it. (If any change is required) Thanks. -- Clebert Suconic

[GitHub] activemq-artemis pull request #1465: NOJIRA fix security policy for Netty te...

2017-08-16 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/1465 NOJIRA fix security policy for Netty test You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis master_work

[GitHub] activemq-artemis issue #1464: Implement support for intercepting additional ...

2017-08-16 Thread orpiske
Github user orpiske commented on the issue: https://github.com/apache/activemq-artemis/pull/1464 Sample output from running the modified Interceptor example: ``` server-out:MQTT Interceptor gets called server-out:A MQTT control packet was intercepted CONNACK

[GitHub] activemq-artemis issue #1464: Implement support for intercepting additional ...

2017-08-16 Thread orpiske
Github user orpiske commented on the issue: https://github.com/apache/activemq-artemis/pull/1464 Some additional comments: @jbertram suggested to also look at org.apache.activemq.artemis.core.protocol.core.impl.ChannelImpl#invokeInterceptors. While I agree that it seems

[GitHub] activemq-artemis pull request #1464: Implement support for intercepting addi...

2017-08-16 Thread orpiske
GitHub user orpiske opened a pull request: https://github.com/apache/activemq-artemis/pull/1464 Implement support for intercepting additional MQTT control packets Previously, only the PUBLISH packet was intercepted. This patch modifies the code to add support for the other