[GitHub] activemq-artemis pull request #1850: init pagingFactory with journalBufferTi...

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

[GitHub] activemq-artemis pull request #1850: init pagingFactory with journalBufferTi...

jbertram-2
GitHub user wy96f opened a pull request:

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

    init pagingFactory with journalBufferTimeout_NIO

    PagingFactory is wrongly initialized with journalBufferSize_NIO not journalBufferTimeout_NIO.

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

    $ git pull https://github.com/wy96f/activemq-artemis buffer_timeout

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

    https://github.com/apache/activemq-artemis/pull/1850.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 #1850
   
----
commit 52d79c09fd26c8d4749d7665db8bcff71b722430
Author: yang wei <wy96fyw@...>
Date:   2018-02-05T08:42:23Z

    init pagingFactory with journalBufferTimeout_NIO

----


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

[GitHub] activemq-artemis issue #1850: init pagingFactory with journalBufferTimeout_N...

jbertram-2
Github user gaohoward commented on the issue:

    https://github.com/apache/activemq-artemis/pull/1850
 
    @wy96f  can you do the forced push to trigger a new Jenkins build?


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

[GitHub] activemq-artemis issue #1850: init pagingFactory with journalBufferTimeout_N...

jbertram-2
In reply to this post by jbertram-2
Github user franz1981 commented on the issue:

    https://github.com/apache/activemq-artemis/pull/1850
 
    @wy96f Please open a jira for it and use the JIRA id on the commit message :+1:


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

[GitHub] activemq-artemis issue #1850: ARTEMIS-1667 init pagingFactory with journalBu...

jbertram-2
In reply to this post by jbertram-2
Github user jbertram commented on the issue:

    https://github.com/apache/activemq-artemis/pull/1850
 
    Nice catch!


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

[GitHub] activemq-artemis pull request #1850: ARTEMIS-1667 init pagingFactory with jo...

jbertram-2
In reply to this post by jbertram-2
Github user asfgit closed the pull request at:

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


---