[GitHub] activemq-artemis pull request #561: ARTEMIS-322 auto-create/delete JMS topic

2016-06-04 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/561 ARTEMIS-322 auto-create/delete JMS topic Implements a new feature for the broker whereby it may automatically create and delete JMS topics which are not explicitly defined through

[GitHub] activemq-artemis issue #557: ARTEMIS-551 Obfuscate truststore password in Tr...

2016-06-03 Thread jbertram
Github user jbertram commented on the issue: https://github.com/apache/activemq-artemis/pull/557 The first line of the commit comment should only be 40 characters long as noted in the Artemis Hacking Guide - https://github.com/apache/activemq-artemis/blob/master/docs/hacking-guide/en

[GitHub] activemq-artemis pull request #556: ARTEMIS-551 Don't log truststore pass

2016-06-03 Thread jbertram
Github user jbertram closed the pull request at: https://github.com/apache/activemq-artemis/pull/556 --- 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 #556: Don't log truststore pass

2016-06-03 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/556 Don't log truststore pass You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis master_work Alternativel

[GitHub] activemq-artemis pull request #:

2016-06-02 Thread jbertram
Github user jbertram commented on the pull request: https://github.com/apache/activemq-artemis/commit/927edb76f323885a7855a7c6fddc1e557a004023#commitcomment-17719306 See PR #554 --- 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 #554: ARTEMIS-386 fix shebang in service scrip...

2016-06-02 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/554 ARTEMIS-386 fix shebang in service script You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis ARTEMIS-386

[GitHub] activemq-artemis pull request #:

2016-06-01 Thread jbertram
Github user jbertram commented on the pull request: https://github.com/apache/activemq-artemis/commit/927edb76f323885a7855a7c6fddc1e557a004023#commitcomment-17699805 Yes, it should. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] activemq-artemis pull request: ARTEMIS-531 fix war name

2016-05-24 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/536 ARTEMIS-531 fix war name You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis master_work Alternatively you

[GitHub] activemq-artemis pull request: ARTEMIS-532 update JMX RMI port pro...

2016-05-24 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/535 ARTEMIS-532 update JMX RMI port prop 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 pull request: ARTEMIS-533 & ARTEMIS-531

2016-05-23 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/532 ARTEMIS-533 & ARTEMIS-531 You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis master_work Alternatively

[GitHub] activemq-artemis pull request: Rework journal reclaimer

2016-05-23 Thread jbertram
Github user jbertram commented on the pull request: https://github.com/apache/activemq-artemis/pull/531#issuecomment-221057286 Can you provide some context and rationale for this change? --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] activemq-artemis pull request: Text fixes + CLI improvement

2016-05-20 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/529 Text fixes + CLI improvement You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis master_work Alternatively

[GitHub] activemq-artemis pull request: ARTEMIS-521 - OSGi support - add un...

2016-05-19 Thread jbertram
Github user jbertram commented on the pull request: https://github.com/apache/activemq-artemis/pull/526#issuecomment-220329595 Apparently org.apache.activemq.artemis.karaf.ArtemisFeatureTest.test failed. Can you look into this, @dejanb? --- If your project is set up for it, you can

[GitHub] activemq-artemis pull request: new logging.properties and trace fo...

2016-05-18 Thread jbertram
Github user jbertram commented on the pull request: https://github.com/apache/activemq-artemis/pull/523#issuecomment-220125612 IMO it's a bit verbose for normal test logging, but I can see how it might be useful for actual traces. --- If your project is set up for it, you can

[GitHub] activemq-artemis pull request: ARTEMIS-523 re-order SASL mechs

2016-05-17 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/521 ARTEMIS-523 re-order SASL mechs You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis ARTEMIS-523 Alternatively

[GitHub] activemq-artemis pull request: ARTEMIS-522 fix AMQP ANON advert

2016-05-17 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/519 ARTEMIS-522 fix AMQP ANON advert You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis ARTEMIS-522

[GitHub] activemq-artemis pull request: ARTEMIS-519 protocol-manager-factor...

2016-05-16 Thread jbertram
Github user jbertram commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/517#discussion_r63352943 --- Diff: artemis-jms-server/src/main/java/org/apache/activemq/artemis/jms/management/impl/JMSConnectionFactoryControlImpl.java --- @@ -432,6

[GitHub] activemq-artemis pull request: ARTEMIS-517

2016-05-10 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/513 ARTEMIS-517 You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis ARTEMIS-517 Alternatively you can review and

[GitHub] activemq-artemis pull request: Make ScaleDownTest#testStoreAndForw...

2016-05-06 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/511 Make ScaleDownTest#testStoreAndForward more robust You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis

[GitHub] activemq-artemis pull request: Update auto-create/delete tests

2016-05-05 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/510 Update auto-create/delete tests You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis master_work Alternatively

[GitHub] activemq-artemis pull request: Auto-delete JMS, not just core

2016-05-05 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/509 Auto-delete JMS, not just core You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis master_work Alternatively

[GitHub] activemq-artemis pull request: Improve packet toString() to ease d...

2016-05-05 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/508 Improve packet toString() to ease debug You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis packetToString

[GitHub] activemq-artemis pull request: Fix InterceptorTest

2016-05-05 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/507 Fix InterceptorTest You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis master_work Alternatively you can

[GitHub] activemq-artemis pull request: More test fixes

2016-05-04 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/505 More test fixes You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis master_work Alternatively you can review

[GitHub] activemq-artemis pull request: ARTEMIS-512 fix variable name confl...

2016-05-04 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/503 ARTEMIS-512 fix variable name conflict 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 pull request: ARTEMIS-512 NPEs in LDAP authz

2016-05-02 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/500 ARTEMIS-512 NPEs in LDAP authz You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis ARTEMIS-512 Alternatively

[GitHub] activemq-artemis pull request: ARTEMIS-510 create queue only on ro...

2016-04-29 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/498 ARTEMIS-510 create queue only on route You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis ARTEMIS-510

[GitHub] activemq-artemis pull request: ARTEMIS-459 NPE during RA tearDown

2016-04-26 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/491 ARTEMIS-459 NPE during RA tearDown You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis ARTEMIS-459

[GitHub] activemq-artemis pull request: ARTEMIS-386 fix shebang

2016-04-25 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/489 ARTEMIS-386 fix shebang You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis ARTEMIS-386 Alternatively you can

[GitHub] activemq-artemis pull request: ARTEMIS-417 more broker-level JMX a...

2016-04-19 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/473 ARTEMIS-417 more broker-level JMX attrs You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis ARTEMIS-417

[GitHub] activemq-artemis pull request: ARTEMIS-489 update examples doc

2016-04-18 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/470 ARTEMIS-489 update examples doc You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis ARTEMIS-489 Alternatively

[GitHub] activemq-artemis pull request: ARTEMIS-400 allow SSL store reload

2016-04-15 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/467 ARTEMIS-400 allow SSL store reload You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis ARTEMIS-400

[GitHub] activemq-artemis pull request: ARTEMIS-407 remove StompConnect ref...

2016-04-15 Thread jbertram
Github user jbertram commented on the pull request: https://github.com/apache/activemq-artemis/pull/466#issuecomment-210510297 Local PR check succeeded. I don't know what's going on with the Apache PR build. --- If your project is set up for it, you can reply to this emai

[GitHub] activemq-artemis pull request: ARTEMIS-407 remove StompConnect ref...

2016-04-15 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/466 ARTEMIS-407 remove StompConnect references You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis ARTEMIS-407

[GitHub] activemq-artemis pull request: ARTEMIS-405 JMX attributes doc

2016-04-15 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/464 ARTEMIS-405 JMX attributes doc You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis ARTEMIS-405 Alternatively

[GitHub] activemq-artemis pull request: ARTEMIS-474 and ARTEMIS-484

2016-04-14 Thread jbertram
Github user jbertram commented on the pull request: https://github.com/apache/activemq-artemis/pull/463#issuecomment-210177993 Looks good. Nice work! --- 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 pull request: ARTEMIS-474 and ARTEMIS-484

2016-04-14 Thread jbertram
Github user jbertram commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/463#discussion_r59800041 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/replication/ReplicationEndpoint.java --- @@ -80,13 +80,17 @@ import

[GitHub] activemq-artemis pull request: ARTEMIS-420 user switch when broker...

2016-04-13 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/461 ARTEMIS-420 user switch when broker is service You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis ARTEMIS-420

[GitHub] activemq-artemis pull request: ARTEMIS-476 example fixes/docs

2016-04-13 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/460 ARTEMIS-476 example fixes/docs You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis ARTEMIS-476 Alternatively

[GitHub] activemq-artemis pull request: ARTEMIS-471 NPE during replication

2016-04-07 Thread jbertram
Github user jbertram closed the pull request at: https://github.com/apache/activemq-artemis/pull/441 --- 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: ARTEMIS-471 NPE during replication

2016-04-06 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/441 ARTEMIS-471 NPE during replication You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis ARTEMIS-471

[GitHub] activemq-artemis pull request: ARTEMIS-467 Potential NPE

2016-04-05 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/439 ARTEMIS-467 Potential NPE You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis ARTEMIS-467 Alternatively you

[GitHub] activemq-artemis pull request: ARTEMIS-462 JMSXGroupID on JMS 2 pr...

2016-03-30 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/433 ARTEMIS-462 JMSXGroupID on JMS 2 producer You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis ARTEMIS-462

[GitHub] activemq-artemis pull request: ARTEMIS-451 JAAS user/role props re...

2016-03-22 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/427 ARTEMIS-451 JAAS user/role props reload These are from ActiveMQ 5.x, specifically https://issues.apache.org/jira/browse/AMQ-6214 and https://issues.apache.org/jira/browse/AMQ-5876. You

[GitHub] activemq-artemis pull request: ARTEMIS-445 avoid NPE on null ks pa...

2016-03-21 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/426 ARTEMIS-445 avoid NPE on null ks password You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis ARTEMIS-445

[GitHub] activemq-artemis pull request: ARTEMIS-441 correct time-unit for l...

2016-03-20 Thread jbertram
Github user jbertram commented on the pull request: https://github.com/apache/activemq-artemis/pull/423#issuecomment-197934280 I ran the PR build locally and it was successful. I'm not sure what's going on with the automated one. --- If your project is set up for it, you

[GitHub] activemq-artemis pull request: ARTEMIS-441 correct time-unit for l...

2016-03-19 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/423 ARTEMIS-441 correct time-unit for large msg poll You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis ARTEMIS

[GitHub] activemq-artemis pull request: ARTEMIS-432 client gets msg after s...

2016-03-11 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/420 ARTEMIS-432 client gets msg after session stop You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis ARTEMIS-432

[GitHub] activemq-artemis pull request: ARTEMIS-385 clean-up factory on tim...

2016-03-03 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/410 ARTEMIS-385 clean-up factory on timeout This restores a call to ClientSessionFactory#cleanup that appears to have been mistakenly removed by a previous change related to this JIRA

[GitHub] activemq-artemis pull request: ARTEMIS-422 support appendRollbackR...

2016-02-25 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/406 ARTEMIS-422 support appendRollbackRecord You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis ARTEMIS-422

[GitHub] activemq-artemis pull request: ARTEMIS-350 fix for potential race

2016-02-23 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/400 ARTEMIS-350 fix for potential race It's possible for the latch used for flow control here to get out of sync. In other words, multiple count-downs can occur between count-ups so

[GitHub] activemq-artemis pull request: ARTEMIS-391 log WARN on cxn limit

2016-02-17 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/392 ARTEMIS-391 log WARN on cxn limit You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis ARTEMIS-391

[GitHub] activemq-artemis pull request: ARTEMIS-399 support 'clientId' acti...

2016-02-16 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/391 ARTEMIS-399 support 'clientId' activation prop You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis A

[GitHub] activemq-artemis pull request: ARTEMIS-395 auto-delete only if set...

2016-02-09 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/384 ARTEMIS-395 auto-delete only if setting is true You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis ARTEMIS

[GitHub] activemq-artemis pull request: ARTEMIS-388 listen for activation f...

2016-02-08 Thread jbertram
Github user jbertram commented on the pull request: https://github.com/apache/activemq-artemis/pull/381#issuecomment-181617040 @clebertsuconic, yes. That's precisely the use-case I was targeting. Note the test-case I added tests this explicitly. --- If your project is set u

[GitHub] activemq-artemis pull request: ARTEMIS-388 listen for activation f...

2016-02-08 Thread jbertram
Github user jbertram commented on the pull request: https://github.com/apache/activemq-artemis/pull/381#issuecomment-181611986 @clebertsuconic, can you elaborate a bit more on that use-case? I'm not following. To what does a NettyAcceptor connect? --- If your project is set u

[GitHub] activemq-artemis pull request: ARTEMIS-388 listen for activation f...

2016-02-08 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/381 ARTEMIS-388 listen for activation failures You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis ARTEMIS-388

[GitHub] activemq-artemis pull request: Update the resteasy version

2016-02-08 Thread jbertram
Github user jbertram commented on the pull request: https://github.com/apache/activemq-artemis/pull/375#issuecomment-181462442 Can you rebase this so the PR build will kick-off again? I'm not sure why it failed. --- If your project is set up for it, you can reply to this

[GitHub] activemq-artemis pull request: ARTEMIS-387 grammar fix

2016-02-05 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/373 ARTEMIS-387 grammar fix You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis ARTEMIS-387 Alternatively you can

[GitHub] activemq-artemis pull request: ARTEMIS-376 NPE during fail-back

2016-02-04 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/372 ARTEMIS-376 NPE during fail-back You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis ARTEMIS-376

[GitHub] activemq-artemis pull request: ARTEMIS-385 fixing race

2016-02-03 Thread jbertram
Github user jbertram commented on the pull request: https://github.com/apache/activemq-artemis/pull/369#issuecomment-179587849 @clebertsuconic, I certainly think it's possible. However, I haven't dug into that issue much as I've been working on other tasks so I can&

[GitHub] activemq-artemis pull request: ARTEMIS-374 support schedule messag...

2016-02-01 Thread jbertram
Github user jbertram commented on the pull request: https://github.com/apache/activemq-artemis/pull/362#issuecomment-178061415 I'm not following. From what I can tell in addHead() (where I need to do the check) Rreference.getScheduledDeliveryTime() will always return 0 so there

[GitHub] activemq-artemis pull request: ARTEMIS-374 support schedule messag...

2016-02-01 Thread jbertram
Github user jbertram commented on the pull request: https://github.com/apache/activemq-artemis/pull/362#issuecomment-178012905 I need some way to differentiate messages that are put on the head of the queue by the scheduler and those that aren't because they need to be tr

[GitHub] activemq-artemis pull request: ARTEMIS-374 support schedule messag...

2016-01-29 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/362 ARTEMIS-374 support schedule messages on LVQ You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis LVQ2

[GitHub] activemq-artemis pull request: Fix broken LVQ example

2016-01-27 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/358 Fix broken LVQ example You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis master_work Alternatively you can

[GitHub] activemq-artemis pull request: ARTEMIS-357 fixing an issue created...

2016-01-26 Thread jbertram
Github user jbertram commented on the pull request: https://github.com/apache/activemq-artemis/pull/350#issuecomment-175378662 Nice work, @clebertsuconic. Glad we squashed this bug. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] activemq-artemis pull request: ARTEMIS-358 topic mistakenly remove...

2016-01-25 Thread jbertram
Github user jbertram commented on the pull request: https://github.com/apache/activemq-artemis/pull/346#issuecomment-174735323 Not sure why this was reported as failed. Everything looks good to me: https://builds.apache.org/job/ActiveMQ-Artemis-PR-Build/962/console. --- If your

[GitHub] activemq-artemis pull request: ARTEMIS-358 topic mistakenly remove...

2016-01-25 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/346 ARTEMIS-358 topic mistakenly removed with sub The problem here is that the management notification listener was mistakenly removing the topic itself instead of just the non-durable

[GitHub] activemq-artemis pull request: ARTEMIS-358 topic mistakenly remove...

2016-01-25 Thread jbertram
Github user jbertram commented on the pull request: https://github.com/apache/activemq-artemis/pull/346#issuecomment-174705757 I think @howardgao should take a look at this before it's merged. --- If your project is set up for it, you can reply to this email and have your

[GitHub] activemq-artemis pull request: ARTEMIS-356 - better support for sp...

2016-01-22 Thread jbertram
Github user jbertram commented on the pull request: https://github.com/apache/activemq-artemis/pull/336#issuecomment-173968186 Checkstyle failure: [ERROR] src/main/java/org/apache/activemq/artemis/selector/filter/ComparisonExpression.java[94:13] (blocks) RightCurly

[GitHub] activemq-artemis pull request: ARTEMIS-354 set login.config for Wi...

2016-01-21 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/335 ARTEMIS-354 set login.config for Win service 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 pull request: Fix JMS queue doc

2016-01-20 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/332 Fix JMS queue doc You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis master_work Alternatively you can

[GitHub] activemq-artemis pull request: ARTEMIS-352 security example readme...

2016-01-19 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/328 ARTEMIS-352 security example readme fix You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis ARTEMIS-352

[GitHub] activemq-artemis pull request: Ldap event listener + 'stop' comman...

2016-01-18 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/325 Ldap event listener + 'stop' command fix You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis ldapEve

[GitHub] activemq-artemis pull request: ARTEMIS-336 Windows script updates

2016-01-13 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/318 ARTEMIS-336 Windows script updates 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 pull request: ARTEMIS-336 don't hard-code instanc...

2016-01-11 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/309 ARTEMIS-336 don't hard-code instance path You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis master

[GitHub] activemq-artemis pull request: ARTEMIS-333 Redistribution + groups...

2016-01-06 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/297 ARTEMIS-333 Redistribution + groups tests failing You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis

[GitHub] activemq-artemis pull request: ARTEMIS-218 auto-create in cluster

2016-01-05 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/296 ARTEMIS-218 auto-create in cluster I'm not 100% certain this is the problem described on ARTEMIS-218, but I did find a problem with auto-creating JMS queues in a cluster and

[GitHub] activemq-artemis pull request: fix security doc

2016-01-05 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/294 fix security doc You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis master_work Alternatively you can review

[GitHub] activemq-artemis pull request: ARTEMIS-331 support 0-length large ...

2016-01-05 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/293 ARTEMIS-331 support 0-length large msg You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis ARTEMIS-331

[GitHub] activemq-artemis pull request: ARTEMIS-177 create/destroy subscrip...

2016-01-05 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/292 ARTEMIS-177 create/destroy subscription race You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis ARTEMIS-177

[GitHub] activemq-artemis pull request: ARTEMIS-201 warn of potential OOME

2015-11-23 Thread jbertram
Github user jbertram commented on the pull request: https://github.com/apache/activemq-artemis/pull/245#issuecomment-159054551 The scenario you outline (X addresses with Y max-size-bytes and Z heap-size that would require N addresses to go into paging mode before an OOME occurs) is

[GitHub] activemq-artemis pull request: ARTEMIS-201 warn of potential OOME

2015-11-23 Thread jbertram
Github user jbertram commented on the pull request: https://github.com/apache/activemq-artemis/pull/245#issuecomment-159003967 Even though they have monitoring in place to avoid the OOME, the OOME is still possible if something were to malfunction with regards to the monitoring. I

[GitHub] activemq-artemis pull request: ARTEMIS-239 disable ttl for STOMP c...

2015-11-18 Thread jbertram
Github user jbertram commented on the pull request: https://github.com/apache/activemq-artemis/pull/208#issuecomment-157844491 @themerius, I'm not sure if @clebertsuconic is done with the test changes, but in any case you can add @jscheid as a git remote and cherry-pick his c

[GitHub] activemq-artemis pull request: ARTEMIS-286 Upgrade Netty to versio...

2015-10-28 Thread jbertram
Github user jbertram commented on the pull request: https://github.com/apache/activemq-artemis/pull/221#issuecomment-151905257 Two questions: 1) Is there a particular reason for this upgrade aside from simply keeping up with the latest release? 2) Can you link to the

[GitHub] activemq-artemis pull request: ARTEMIS-256 orchestrate failback de...

2015-10-20 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/204 ARTEMIS-256 orchestrate failback deterministically The failback process needs to be deterministic rather than relying on various incarnations of Thread.sleep() at crucial points

[GitHub] activemq-artemis pull request: ARTEMIS-255 make non-blocking-failo...

2015-10-12 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/195 ARTEMIS-255 make non-blocking-failover-timeout configurable Use the callFailoverTimeout for both blocking and non-blocking calls. You can merge this pull request into a Git repository by

[GitHub] activemq-artemis pull request: ARTEMIS-254 don't throw j.l.ISE fro...

2015-10-12 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/194 ARTEMIS-254 don't throw j.l.ISE from producer You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis ARTEMI

[GitHub] activemq-artemis pull request: ARTEMIS-74 import JAAS auth from 5....

2015-10-07 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/188 ARTEMIS-74 import JAAS auth from 5.x This change allows the use of JAAS login modules for basic authentication and authorization. You can merge this pull request into a Git

[GitHub] activemq-artemis pull request: Adding jmeter-tests and instruction...

2015-10-06 Thread jbertram
Github user jbertram commented on the pull request: https://github.com/apache/activemq-artemis/pull/186#issuecomment-146000651 Would probably be worth squashing all the commits down to one as well. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] activemq-artemis pull request: ARTEMIS-241 - versioning wrong and ...

2015-10-05 Thread jbertram
Github user jbertram commented on the pull request: https://github.com/apache/activemq-artemis/pull/185#issuecomment-145614212 I'm not sure I understand why it's necessary to change over 3,000 files just to demonstrate an OpenWire exception. Can you clarify this point? -

[GitHub] activemq-artemis pull request: Clarify HTTP(S) upgrade config

2015-10-05 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/184 Clarify HTTP(S) upgrade config Demonstrate that the Artemis acceptor receiving HTTP(S) upgrade connections doesn't need any SSL configuration. You can merge this pull request i

[GitHub] activemq-artemis pull request: ARTEMIS-225 validate clientID is se...

2015-09-17 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/171 ARTEMIS-225 validate clientID is set for durable sub You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis

[GitHub] activemq-artemis pull request: ARTEMIS-224 add logmanager to examp...

2015-09-17 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/170 ARTEMIS-224 add logmanager to example classpaths You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis ARTEMIS

[GitHub] activemq-artemis pull request: Count references for RA connection ...

2015-09-14 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/169 Count references for RA connection factories You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis raFix

[GitHub] activemq-artemis pull request: ARTEMIS-211 Memory leak using wildc...

2015-09-02 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/150 ARTEMIS-211 Memory leak using wildcard topic You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis ARTEMIS-211

[GitHub] activemq-artemis pull request: ARTEMIS-206 HTTP Upgrade does not w...

2015-08-31 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/147 ARTEMIS-206 HTTP Upgrade does not work over HTTPS You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis ARTEMIS

[GitHub] activemq-artemis pull request: ARTEMIS-212 Unable to parse IPv6 ad...

2015-08-27 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/140 ARTEMIS-212 Unable to parse IPv6 address You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis ARTEMIS-212

[GitHub] activemq-artemis pull request: ARTEMIS-210 outbound RA connection ...

2015-08-24 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/133 ARTEMIS-210 outbound RA connection load-balancing Inbound sessions are always created from the same ActiveMQConnectionFactory which means the load-balancing policy is applied to them

[GitHub] activemq-artemis pull request: Couple of fixes

2015-08-14 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/130 Couple of fixes You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis master_work Alternatively you can review

<    5   6   7   8   9   10   11   >