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

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

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

pgfox
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 <[hidden email]>
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.

----


---
Reply | Threaded
Open this post in threaded view
|

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

pgfox
Github user asfgit closed the pull request at:

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


---
Reply | Threaded
Open this post in threaded view
|

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

pgfox
In reply to this post by pgfox
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.


---