[GitHub] activemq-artemis issue #1742: ARTEMIS-1570 Flush appendExecutor before take ...

2018-01-24 Thread shoukunhuai
Github user shoukunhuai commented on the issue:

https://github.com/apache/activemq-artemis/pull/1742
  
@clebertsuconic sorry, almost forgot this, being busy on something else.
Looks like this issue was resolved?


---


[GitHub] activemq-artemis issue #1733: [DO NOT MERGE] ARTEMIS-1545 Support JMS 2.0 Co...

2018-01-24 Thread michaelandrepearce
Github user michaelandrepearce commented on the issue:

https://github.com/apache/activemq-artemis/pull/1733
  
Also i can’t remember (sorry laptop is no where near me tonight taking 
some time out tonight) but I remember issue in Bridge test but I was 
concentrating on this first


---


[GitHub] activemq-artemis issue #1733: [DO NOT MERGE] ARTEMIS-1545 Support JMS 2.0 Co...

2018-01-24 Thread michaelandrepearce
Github user michaelandrepearce commented on the issue:

https://github.com/apache/activemq-artemis/pull/1733
  
I saw it also on the journal one as well.


---


[GitHub] activemq-artemis issue #1733: [DO NOT MERGE] ARTEMIS-1545 Support JMS 2.0 Co...

2018-01-24 Thread clebertsuconic
Github user clebertsuconic commented on the issue:

https://github.com/apache/activemq-artemis/pull/1733
  
.. just to say I'm working on it.. getting my head and debugging it..


---


[GitHub] activemq-artemis issue #1733: [DO NOT MERGE] ARTEMIS-1545 Support JMS 2.0 Co...

2018-01-24 Thread clebertsuconic
Github user clebertsuconic commented on the issue:

https://github.com/apache/activemq-artemis/pull/1733
  
@michaelandrepearce the error you saw was a LargeMessage test dead locking 
under JDBC for no apparent connected reason?

if that's the case.. I am working on it now. just confirm please?


---


[GitHub] activemq-artemis issue #1733: [DO NOT MERGE] ARTEMIS-1545 Support JMS 2.0 Co...

2018-01-24 Thread clebertsuconic
Github user clebertsuconic commented on the issue:

https://github.com/apache/activemq-artemis/pull/1733
  
@michaelandrepearce I rebased and I'm running tests to get a current 
status..


---


[GitHub] activemq-artemis issue #1796: ARTEMIS-1623 ActiveMQServerPlugin impl for log...

2018-01-24 Thread pgfox
Github user pgfox commented on the issue:

https://github.com/apache/activemq-artemis/pull/1796
  
@clebertsuconic no problem Clebert. I will create a new PR when I have 
those pending things completed.   


---


[GitHub] activemq-artemis issue #1796: ARTEMIS-1623 ActiveMQServerPlugin impl for log...

2018-01-24 Thread clebertsuconic
Github user clebertsuconic commented on the issue:

https://github.com/apache/activemq-artemis/pull/1796
  
@pgfox I had missed there were still pending things.. I read it wrongly.
Can you send another PR? or should I revert and you sending it again?


---


[GitHub] activemq-artemis issue #1796: ARTEMIS-1623 ActiveMQServerPlugin impl for log...

2018-01-24 Thread pgfox
Github user pgfox commented on the issue:

https://github.com/apache/activemq-artemis/pull/1796
  
@clebertsuconic  
There was some outstanding improvements suggested by @jbertram 
@michaelandrepearce  (above) on this PR that I have not finish yet. 

Can I just open a new PR for those improvement when I have them completed? 
I should have them done in the next day or so. Sorry, for any confusion this 
has caused.





---


[GitHub] activemq-artemis issue #1796: ARTEMIS-1623 ActiveMQServerPlugin impl for log...

2018-01-24 Thread pgfox
Github user pgfox commented on the issue:

https://github.com/apache/activemq-artemis/pull/1796
  
 I still had to do the suggested changes we discussed :( 

Sorry I did not think the would be merged yet. I hope to get those changes 
pushed to the PR in the next day or so.


 


---


[GitHub] activemq-artemis pull request #1815: [ARTEMIS-1030] document url escaping of...

2018-01-24 Thread gtully
GitHub user gtully opened a pull request:

https://github.com/apache/activemq-artemis/pull/1815

[ARTEMIS-1030] document url escaping of the virtualTopicConsumerWildc…

…ards value

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/gtully/activemq-artemis ARTEMIS-1030

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/activemq-artemis/pull/1815.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1815


commit 23b3401bb067fa9f2d66e418a2b4e245dc8177e4
Author: gtully 
Date:   2018-01-24T14:20:05Z

[ARTEMIS-1030] document url escaping of the virtualTopicConsumerWildcards 
value




---


[GitHub] activemq-artemis pull request #1796: ARTEMIS-1623 ActiveMQServerPlugin impl ...

2018-01-24 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/activemq-artemis/pull/1796


---


[GitHub] activemq-artemis pull request #1814: ARTEMIS-1632 Upgrade JBoss logging to 3...

2018-01-24 Thread dejanb
GitHub user dejanb opened a pull request:

https://github.com/apache/activemq-artemis/pull/1814

ARTEMIS-1632 Upgrade JBoss logging to 3.3.1.Final



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/dejanb/activemq-artemis master

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/activemq-artemis/pull/1814.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1814


commit 2aa0996b4f83fab3e1d922016df9570a22b63c75
Author: Dejan Bosanac 
Date:   2018-01-24T10:50:22Z

ARTEMIS-1632 Upgrade JBoss logging to 3.3.1.Final




---


[GitHub] activemq-artemis pull request #1813: [ARTEMIS-1630] Create Artemis Client fe...

2018-01-24 Thread johnpoth
GitHub user johnpoth opened a pull request:

https://github.com/apache/activemq-artemis/pull/1813

[ARTEMIS-1630] Create Artemis Client features

[ARTEMIS-1630](https://issues.apache.org/jira/browse/ARTEMIS-1630)

This adds support for CORE, JMS and AMQP artemis client karaf features with 
some IT tests. Let me know if you have any questions.

Thanks!

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/johnpoth/activemq-artemis ARTEMIS-1630

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/activemq-artemis/pull/1813.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1813


commit cef1cc8f8df58f6121e8d8256edeaef365ae056e
Author: jpoth 
Date:   2018-01-23T11:18:07Z

[ARTEMIS-1630] Create Artemis Client features




---