[GitHub] activemq-artemis pull request #1619: ARTEMIS-1447 JDBC NodeManager to suppor...

[franz1981 <gi...@git.apache.org>]

GitHub user franz1981 opened a pull request:

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

    ARTEMIS-1447 JDBC NodeManager to support JDBC HA Shared Store

    It fixes the NPE on server start due to:
     - missing SqlProviderFactory
      -missing executor factory/scheduled pool (ie using exclusive scheduled pools)
    
    It fixes the WARNINGS due to wrong slowness detection while renewing JdbcLeaseLock.

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

    $ git pull https://github.com/franz1981/activemq-artemis jdbc_ha_fix

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

    https://github.com/apache/activemq-artemis/pull/1619.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 #1619
    
----
commit 152e5f4039a9b7607140e620220cbee9308dc25e
Author: Francesco Nigro <ni...@gmail.com>
Date:   2017-10-27T15:43:29Z

    ARTEMIS-1447 JDBC NodeManager to support JDBC HA Shared Store
    
    It fixes the NPE on server start due to:
     - missing SqlProviderFactory
      -missing executor factory/scheduled pool (ie using exclusive scheduled pools)
    
    It fixes the WARNINGS due to wrong slowness detection while renewing JdbcLeaseLock.

----


---

[GitHub] activemq-artemis issue #1619: ARTEMIS-1447 JDBC NodeManager to support JDBC ...

[clebertsuconic <gi...@git.apache.org>]

Github user clebertsuconic commented on the issue:

    https://github.com/apache/activemq-artemis/pull/1619
  
    I have removed that constructor... I have run the whole testsuite.. it's all good.


---

[GitHub] activemq-artemis pull request #1619: ARTEMIS-1447 JDBC NodeManager to suppor...

[asfgit <gi...@git.apache.org>]

Github user asfgit closed the pull request at:

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


---