[GitHub] activemq-artemis pull request #711: ARTEMIS-601 Implementing reload manager ...

GitHub user clebertsuconic opened a pull request: https://github.com/apache/activemq-artemis/pull/711 ARTEMIS-601 Implementing reload manager on JMS Destinations and Address Settings You can merge this pull request into a Git repository by running: $ git pull

[GitHub] activemq-artemis issue #705: ARTEMIS-670 Adding destination creation and del...

Github user gaohoward commented on the issue: https://github.com/apache/activemq-artemis/pull/705 @clebertsuconic ok looks like the sub-command style (like data command) has no objection. I'll go for it. Thanks --- If your project is set up for it, you can reply to this email and

[GitHub] activemq-artemis issue #632: ARTEMIS-604 - Message Serialization Improvement

Github user gaohoward commented on the issue: https://github.com/apache/activemq-artemis/pull/632 Closing it as we'll have a new one. --- 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 pull request #632: ARTEMIS-604 - Message Serialization Impr...

Github user gaohoward closed the pull request at: https://github.com/apache/activemq-artemis/pull/632 --- 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 feature enabled and wishes so, or if the

[GitHub] activemq-artemis pull request #708: Misc trivial fixes

Github user asfgit closed the pull request at: https://github.com/apache/activemq-artemis/pull/708 --- 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 feature enabled and wishes so, or if the

[GitHub] activemq-artemis pull request #706: ARTEMIS-671 Returning messages after con...

Github user asfgit closed the pull request at: https://github.com/apache/activemq-artemis/pull/706 --- 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 feature enabled and wishes so, or if the

[GitHub] activemq-artemis pull request #707: Artemis 604 Serialization Improvements

Github user asfgit closed the pull request at: https://github.com/apache/activemq-artemis/pull/707 --- 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 feature enabled and wishes so, or if the

[GitHub] activemq-artemis pull request #709: ARTEMIS-672 Use sh not bash in scripts

GitHub user mtaylor opened a pull request: https://github.com/apache/activemq-artemis/pull/709 ARTEMIS-672 Use sh not bash in scripts You can merge this pull request into a Git repository by running: $ git pull https://github.com/mtaylor/activemq-artemis ARTEMIS-672

[GitHub] activemq-artemis issue #709: ARTEMIS-672 Use sh not bash in scripts

Github user mtaylor commented on the issue: https://github.com/apache/activemq-artemis/pull/709 @jdanekrh has offered to do some testing on the various flavours of linux. Please don't merge until @jdanekrh has ack'd here. --- If your project is set up for it, you can reply to this

[GitHub] activemq-artemis issue #548: Executor in JournalImpl for append operations

Github user mtaylor commented on the issue: https://github.com/apache/activemq-artemis/pull/548 @clebertsuconic Any update on this? Did you manage to complete the comparison you wanted to do? --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] activemq-artemis issue #706: ARTEMIS-671 Returning messages after connection...

Github user johnament commented on the issue: https://github.com/apache/activemq-artemis/pull/706 BTW -> SOP = System.out.println instead of logging. --- 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

[GitHub] activemq-artemis issue #705: ARTEMIS-670 Adding destination creation and del...

Github user gaohoward commented on the issue: https://github.com/apache/activemq-artemis/pull/705 @clebertsuconic ok done. --- 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 feature

[GitHub] activemq-artemis pull request #707: Artemis 604 Serialization Improvements

GitHub user mtaylor opened a pull request: https://github.com/apache/activemq-artemis/pull/707 Artemis 604 Serialization Improvements You can merge this pull request into a Git repository by running: $ git pull https://github.com/mtaylor/activemq-artemis ARTEMIS-604

[GitHub] activemq-artemis pull request #704: ARTEMIS-669 Do binding query on sender l...

Github user asfgit closed the pull request at: https://github.com/apache/activemq-artemis/pull/704 --- 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 feature enabled and wishes so, or if the

[GitHub] activemq-artemis pull request #708: Misc trivial fixes

GitHub user scop opened a pull request: https://github.com/apache/activemq-artemis/pull/708 Misc trivial fixes You can merge this pull request into a Git repository by running: $ git pull https://github.com/scop/activemq-artemis misc Alternatively you can review and apply

[GitHub] activemq-artemis issue #548: Executor in JournalImpl for append operations

Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/548 @mtaylor there's a test failure I need to take care before can 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 #705: ARTEMIS-670 Adding destination creation and del...

Github user gaohoward commented on the issue: https://github.com/apache/activemq-artemis/pull/705 @clebertsuconic yeah we can combine the delete-topic and delete-queue, but having them in separate commands is not bad. I really don't have any preferences of one over the other. ---

[GitHub] activemq-artemis issue #706: ARTEMIS-671 Returning messages after connection...

Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/706 @johnament I see.. it was intentionally SOP then... it would be on junit reports... it was from the test logger.. but it's been removed now. --- If your project is set up for it,

[GitHub] activemq-artemis issue #705: ARTEMIS-670 Adding destination creation and del...

Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/705 @gaohoward when I wrote the delete-topic up here was just as an example of separating the commands. I didn't mean to be literal. Do you actually need to differenticate delete-topic

[GitHub] activemq-artemis issue #705: ARTEMIS-670 Adding destination creation and del...

Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/705 @gaohoward It's different.. you had the behaviour specified by properties... which was the confusing part. Grouping is different... you still have separated commands, and

Added unit tests to Apache.NMS.MSMQ and subsequenly fixed a few bugs

Hi, I just submitted a patch as AMQNET-556 adding unit tests to Apache.NMS.MSMQ and subsequenly fixed a few bugs. Regards, Stéphane Ramet

[GitHub] activemq-artemis issue #705: ARTEMIS-670 Adding destination creation and del...

Github user jbertram commented on the issue: https://github.com/apache/activemq-artemis/pull/705 One concern I have is that I can't tell by the names if these commands are operating on JMS resources or core resources. Obviously there's no such thing as a core topic so I assume that

[GitHub] activemq-artemis issue #705: ARTEMIS-670 Adding destination creation and del...

Github user gaohoward commented on the issue: https://github.com/apache/activemq-artemis/pull/705 @jbertram by default create-queue creates jms queues, if you want create a core queue, use --core option, like artemis create-queue --core --name corequeue1 (optionally you can

[GitHub] activemq-artemis issue #705: ARTEMIS-670 Adding destination creation and del...

Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/705 @jbertram maybe we could call the group jms artemis jms create --queue(default) | --topic --name name artemis jms delete --name name (I'm not convinced we need --queue or

[GitHub] activemq-artemis issue #705: ARTEMIS-670 Adding destination creation and del...

Github user gaohoward commented on the issue: https://github.com/apache/activemq-artemis/pull/705 @clebertsuconic that kinda goes back to my original impl, which combine all the functionalities into one command, i.e. ./artemis destination create|delete --name myqueue1

[GitHub] activemq-artemis issue #705: ARTEMIS-670 Adding destination creation and del...

Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/705 @johnament I liked the idea... we can't use create though on the CLI as it's used to create a broker.. although we could use it with a group: ./artemis dest

[GitHub] activemq-artemis pull request #710: ARTEMIS-601 Fixing use of spaces on the ...

GitHub user clebertsuconic opened a pull request: https://github.com/apache/activemq-artemis/pull/710 ARTEMIS-601 Fixing use of spaces on the server and reloading You can merge this pull request into a Git repository by running: $ git pull

[GitHub] activemq-artemis pull request #710: ARTEMIS-601 Fixing use of spaces on the ...

Github user asfgit closed the pull request at: https://github.com/apache/activemq-artemis/pull/710 --- 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 feature enabled and wishes so, or if the

[GitHub] activemq pull request #196: Fixes CAMEL-10226: camel-jms ignores connection ...

GitHub user chirino opened a pull request: https://github.com/apache/activemq/pull/196 Fixes CAMEL-10226: camel-jms ignores connection pool settings in spri… …ng-boot deployment. You can merge this pull request into a Git repository by running: $ git pull