Re: [DISCUSS] ARTEMIS-1009 Pure Messaging Encoding Refactoring Artemis 2.0

2017-03-05 Thread Clebert Suconic
PR just arrived :) https://github.com/apache/activemq-artemis/pull/1064 On Sat, Mar 4, 2017 at 1:10 AM, Clebert Suconic wrote: > @CShanon: I was actually dealing with the exporter/importer today. > > You're right.. the body of the message is for core-only. > > > We could do the following: > > >

[GitHub] activemq-artemis pull request #1064: Pure message encoding on artemis

2017-03-05 Thread clebertsuconic
GitHub user clebertsuconic opened a pull request: https://github.com/apache/activemq-artemis/pull/1064 Pure message encoding on artemis As previously discussed, this will allow messages to be transfered on their original encoding. I foresee a few tests failing due to the cu

[GitHub] activemq-artemis issue #1062: Artemis 1012 rebased artemis 1009

2017-03-05 Thread mtaylor
Github user mtaylor commented on the issue: https://github.com/apache/activemq-artemis/pull/1062 sure. I'll do it tomorrow mate when I get the office :) On 5 Mar 2017 17:12, "clebertsuconic" wrote: > @mtaylor I will rebase this later this

[GitHub] activemq-artemis issue #1062: Artemis 1012 rebased artemis 1009

2017-03-05 Thread clebertsuconic
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/1062 @mtaylor I will rebase this later this week. Can you do me a favour. type this on your branch: ``` git pull --rebase upstream artemis-1009 ``` I did it

Re: [DISCUSS] Coverity Scan for Artemis

2017-03-05 Thread Jiri Danek
I put a new snapshot into coverity yesterday. There was nothing new found and nothing old fixed. The table with component breakdown on the Coverity website is still broken, not sure why. Your request for analysis of Apache ActiveMQ Artemis has been completed successfully. The results are a

[GitHub] activemq-artemis pull request #1063: ARTEMIS-1017 reword error message when ...

2017-03-05 Thread jdanekrh
GitHub user jdanekrh opened a pull request: https://github.com/apache/activemq-artemis/pull/1063 ARTEMIS-1017 reword error message when default destination for produc… …er is not specified You can merge this pull request into a Git repository by running: $ git pull https://

[GitHub] activemq-artemis issue #1062: Artemis 1012 rebased artemis 1009

2017-03-05 Thread mtaylor
Github user mtaylor commented on the issue: https://github.com/apache/activemq-artemis/pull/1062 Clebert I made a couple changes too. can you delete the commits in your branch and merge these please. On 4 Mar 2017 14:53, "clebertsuconic" wrote: > @mtaylor