[GitHub] activemq-artemis pull request #2476: Fix deadlock while getting queue messag...

2018-12-20 Thread wy96f
Github user wy96f closed the pull request at: https://github.com/apache/activemq-artemis/pull/2476 ---

[GitHub] activemq-artemis issue #2466: ARTEMIS-2206 The MQTT consumer reconnection ca...

2018-12-20 Thread onlyMIT
Github user onlyMIT commented on the issue: https://github.com/apache/activemq-artemis/pull/2466 @michaelandrepearce @clebertsuconic Added automated testing. And added locks for sub and unSub operations ---

[GitHub] activemq-artemis pull request #2476: Fix deadlock while getting queue messag...

2018-12-20 Thread wy96f
GitHub user wy96f opened a pull request: https://github.com/apache/activemq-artemis/pull/2476 Fix deadlock while getting queue message count during cleanup ARTEMIS-2123 introduced a deadlock bug. jstack shows: Thread 1: at

[GitHub] activemq-artemis pull request #2474: [ARTEMIS-1536]: Incorrect Journal files...

2018-12-20 Thread ehsavoie
Github user ehsavoie commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2474#discussion_r243354412 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/persistence/impl/journal/JournalStorageManager.java --- @@ -163,13

[GitHub] activemq-artemis issue #2444: ARTEMIS-2186 Large message incomplete when ser...

2018-12-20 Thread jbertram
Github user jbertram commented on the issue: https://github.com/apache/activemq-artemis/pull/2444 @clebertsuconic, @michaelandrepearce, do you guys think this is ready to be merged now? It looks good to me since there is a test and the file is always synced on `releaseResources`.

[GitHub] activemq-artemis pull request #2474: [ARTEMIS-1536]: Incorrect Journal files...

2018-12-20 Thread clebertsuconic
Github user clebertsuconic commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2474#discussion_r243338608 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/persistence/impl/journal/JournalStorageManager.java --- @@

[GitHub] activemq-artemis pull request #2474: [ARTEMIS-1536]: Incorrect Journal files...

2018-12-20 Thread clebertsuconic
Github user clebertsuconic commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2474#discussion_r243338301 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/persistence/impl/journal/JournalStorageManager.java --- @@

[GitHub] activemq-artemis pull request #2474: [ARTEMIS-1536]: Incorrect Journal files...

2018-12-20 Thread ehsavoie
Github user ehsavoie commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2474#discussion_r243331449 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/persistence/impl/journal/JournalStorageManager.java --- @@ -163,13

[GitHub] activemq-artemis pull request #2474: [ARTEMIS-1536]: Incorrect Journal files...

2018-12-20 Thread clebertsuconic
Github user clebertsuconic commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2474#discussion_r243283678 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/persistence/impl/journal/JournalStorageManager.java --- @@

[GitHub] activemq-artemis issue #2475: ARTEMIS-2144 - tx begin failure in ra doesn't ...

2018-12-20 Thread andytaylor
Github user andytaylor commented on the issue: https://github.com/apache/activemq-artemis/pull/2475 don't merge just yet I want to run the whole test suite ---

[GitHub] activemq-artemis pull request #2475: ARTEMIS-2144 - tx begin failure in ra d...

2018-12-20 Thread andytaylor
GitHub user andytaylor opened a pull request: https://github.com/apache/activemq-artemis/pull/2475 ARTEMIS-2144 - tx begin failure in ra doesn't get cleaned up https://issues.apache.org/jira/browse/ARTEMIS-2144 You can merge this pull request into a Git repository by running:

[GitHub] activemq-artemis pull request #2474: [ARTEMIS-1536]: Incorrect Journal files...

2018-12-20 Thread franz1981
Github user franz1981 commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2474#discussion_r243190151 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/persistence/impl/journal/JournalStorageManager.java --- @@ -163,13

[GitHub] activemq-artemis pull request #2474: [ARTEMIS-1536]: Incorrect Journal files...

2018-12-20 Thread ehsavoie
GitHub user ehsavoie opened a pull request: https://github.com/apache/activemq-artemis/pull/2474 [ARTEMIS-1536]: Incorrect Journal filesize calculation where specified size is lest that the block size when using AIO. * If the specified file size is under the fs block size then the