[GitHub] activemq-artemis pull request #2490: V2 196

2019-01-08 Thread gemmellr
Github user gemmellr commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2490#discussion_r245972322 --- Diff: tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/openwire/amq/QueueConsumerPriorityTest.java ---

[GitHub] activemq-artemis pull request #2467: ARTEMIS-2205 Performance improvements o...

2019-01-08 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2467#discussion_r245960681 --- Diff:

[GitHub] activemq-artemis pull request #2493: ARTEMIS-2223 when a new consumer is cre...

2019-01-08 Thread onlyMIT
GitHub user onlyMIT opened a pull request: https://github.com/apache/activemq-artemis/pull/2493 ARTEMIS-2223 when a new consumer is created, no subscription is called. In the 'MQTTTest.testCleanSession()' test method, when a new consumer is created, no subscription is

[GitHub] activemq-artemis pull request #2492: ARTEMIS-2222 why the position remains u...

2019-01-08 Thread CNNJYB
GitHub user CNNJYB opened a pull request: https://github.com/apache/activemq-artemis/pull/2492 ARTEMIS- why the position remains unchanged if ignored is set to true I am a bit confused about this, When CursorIterator:next is called during queue depage, if ignored is set to

[GitHub] activemq-artemis pull request #2491: ARTEMIS-2217 remove state on clean MQTT...

2019-01-07 Thread onlyMIT
Github user onlyMIT commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2491#discussion_r245879352 --- Diff: artemis-protocols/artemis-mqtt-protocol/src/main/java/org/apache/activemq/artemis/core/protocol/mqtt/MQTTSession.java --- @@ -117,14

[GitHub] activemq-artemis pull request #2491: ARTEMIS-2217 remove state on clean MQTT...

2019-01-07 Thread onlyMIT
Github user onlyMIT commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2491#discussion_r245878622 --- Diff: artemis-protocols/artemis-mqtt-protocol/src/main/java/org/apache/activemq/artemis/core/protocol/mqtt/MQTTSession.java --- @@ -117,14

[GitHub] activemq-artemis pull request #2491: ARTEMIS-2217 remove state on clean MQTT...

2019-01-07 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/2491 ARTEMIS-2217 remove state on clean MQTT session disconnect You can merge this pull request into a Git repository by running: $ git pull

[GitHub] activemq-artemis pull request #2467: ARTEMIS-2205 Performance improvements o...

2019-01-07 Thread clebertsuconic
Github user clebertsuconic commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2467#discussion_r245777806 --- Diff: artemis-protocols/artemis-amqp-protocol/src/main/java/org/apache/activemq/artemis/protocol/amqp/proton/AMQPConnectionContext.java

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

2019-01-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/activemq-artemis/pull/2475 ---

[GitHub] activemq-artemis pull request #2490: V2 196

2019-01-07 Thread michaelandrepearce
GitHub user michaelandrepearce opened a pull request: https://github.com/apache/activemq-artemis/pull/2490 V2 196 @franz1981 an alternative so we don't have to have a copy of CopyOnWriteArrayList, it does mean on add or remove consumer we have to invoke toArray which causes a

[GitHub] activemq-artemis pull request #2489: ARTEMIS-2220 Fix PageCursorStressTest::...

2019-01-04 Thread franz1981
GitHub user franz1981 opened a pull request: https://github.com/apache/activemq-artemis/pull/2489 ARTEMIS-2220 Fix PageCursorStressTest::testSimpleCursorWithFilter NPE FakeQueue is not correctly setting the queue on its PageSubscription, leading to fail the test due to NPEs when

[GitHub] activemq-artemis pull request #2484: ARTEMIS-2216 Use a specific executor fo...

2019-01-04 Thread wy96f
Github user wy96f commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2484#discussion_r245252912 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/paging/impl/PagingStoreImpl.java --- @@ -278,21 +293,26 @@ public boolean

[GitHub] activemq-artemis pull request #2481: ARTEMIS-2213 don't expire critical comp...

2019-01-04 Thread wy96f
Github user wy96f closed the pull request at: https://github.com/apache/activemq-artemis/pull/2481 ---

[GitHub] activemq-artemis pull request #2484: ARTEMIS-2216 Use a specific executor fo...

2019-01-04 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2484#discussion_r245239811 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/paging/impl/PagingStoreImpl.java --- @@ -278,21 +293,26 @@

[GitHub] activemq-artemis pull request #2482: ARTEMIS-2214 Cache durable.

2019-01-04 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2482#discussion_r245233815 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/paging/cursor/PagedReferenceImpl.java --- @@ -120,14 +126,16

[GitHub] activemq-artemis pull request #2484: ARTEMIS-2216 Use a specific executor fo...

2019-01-03 Thread franz1981
Github user franz1981 commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2484#discussion_r245224854 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/paging/impl/PagingStoreImpl.java --- @@ -278,21 +293,26 @@ public

[GitHub] activemq-artemis pull request #2482: ARTEMIS-2214 Cache durable in ...

2019-01-03 Thread wy96f
Github user wy96f commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2482#discussion_r245195927 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/paging/cursor/PagedReferenceImpl.java --- @@ -120,6 +128,8 @@ public

[GitHub] activemq-artemis pull request #2484: ARTEMIS-2216 Use a specific executor fo...

2019-01-03 Thread wy96f
Github user wy96f commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2484#discussion_r245183888 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/paging/impl/PagingStoreImpl.java --- @@ -278,21 +293,26 @@ public boolean

[GitHub] activemq-artemis pull request #2484: ARTEMIS-2216 Use a specific executor fo...

2019-01-03 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2484#discussion_r245095523 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/paging/cursor/impl/PageSubscriptionImpl.java --- @@ -1350,7

[GitHub] activemq-artemis pull request #2484: ARTEMIS-2216 Use a specific executor fo...

2019-01-03 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2484#discussion_r245094809 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/paging/impl/PagingStoreImpl.java --- @@ -278,21 +293,26 @@

[GitHub] activemq-artemis pull request #2483: ARTEMIS-2215 largemessage have been con...

2019-01-03 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2483#discussion_r245090041 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/persistence/impl/journal/JournalStorageManager.java --- @@

[GitHub] activemq-artemis pull request #2484: ARTEMIS-2216 Use a specific executor fo...

2019-01-03 Thread franz1981
Github user franz1981 commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2484#discussion_r245048241 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/paging/impl/PagingStoreImpl.java --- @@ -278,21 +293,26 @@ public

[GitHub] activemq-artemis pull request #2488: ARTEMIS-196 Implement Consumer Priority

2019-01-03 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2488#discussion_r245030347 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/QueueConsumersImpl.java --- @@ -0,0 +1,648 @@

[GitHub] activemq-artemis pull request #2488: ARTEMIS-196 Implement Consumer Priority

2019-01-03 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2488#discussion_r245029912 --- Diff:

[GitHub] activemq-artemis pull request #2488: ARTEMIS-196 Implement Consumer Priority

2019-01-03 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2488#discussion_r245029854 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/ResetableIterator.java --- @@ -0,0 +1,29 @@

[GitHub] activemq-artemis pull request #2488: ARTEMIS-196 Implement Consumer Priority

2019-01-03 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2488#discussion_r245029745 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/QueueImpl.java --- @@ -2497,45 +2494,36 @@

[GitHub] activemq-artemis pull request #2488: ARTEMIS-196 Implement Consumer Priority

2019-01-03 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2488#discussion_r245029528 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/QueueConsumersImpl.java --- @@ -0,0 +1,648 @@

[GitHub] activemq-artemis pull request #2488: ARTEMIS-196 Implement Consumer Priority

2019-01-03 Thread franz1981
Github user franz1981 commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2488#discussion_r245014300 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/ResetableIterator.java --- @@ -0,0 +1,29 @@ +/* +

[GitHub] activemq-artemis pull request #2488: ARTEMIS-196 Implement Consumer Priority

2019-01-03 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2488#discussion_r245013090 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/QueueConsumersImpl.java --- @@ -0,0 +1,648 @@

[GitHub] activemq-artemis pull request #2488: ARTEMIS-196 Implement Consumer Priority

2019-01-03 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2488#discussion_r245011975 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/ResetableIterator.java --- @@ -0,0 +1,29 @@

[GitHub] activemq-artemis pull request #2488: ARTEMIS-196 Implement Consumer Priority

2019-01-03 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2488#discussion_r245012214 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/QueueConsumersImpl.java --- @@ -0,0 +1,648 @@

[GitHub] activemq-artemis pull request #2488: ARTEMIS-196 Implement Consumer Priority

2019-01-03 Thread franz1981
Github user franz1981 commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2488#discussion_r245010184 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/QueueConsumersImpl.java --- @@ -0,0 +1,648 @@ +/*

[GitHub] activemq-artemis pull request #2488: ARTEMIS-196 Implement Consumer Priority

2019-01-03 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2488#discussion_r245009418 --- Diff:

[GitHub] activemq-artemis pull request #2488: ARTEMIS-196 Implement Consumer Priority

2019-01-03 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2488#discussion_r245009864 --- Diff: artemis-protocols/artemis-amqp-protocol/src/main/java/org/apache/activemq/artemis/protocol/amqp/broker/AMQPSessionCallback.java

[GitHub] activemq-artemis pull request #2488: ARTEMIS-196 Implement Consumer Priority

2019-01-03 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2488#discussion_r245008661 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/QueueImpl.java --- @@ -3080,45 +3053,20 @@

[GitHub] activemq-artemis pull request #2488: ARTEMIS-196 Implement Consumer Priority

2019-01-03 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2488#discussion_r245009338 --- Diff:

[GitHub] activemq-artemis pull request #2488: ARTEMIS-196 Implement Consumer Priority

2019-01-03 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2488#discussion_r245008000 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/QueueImpl.java --- @@ -2497,45 +2494,36 @@

[GitHub] activemq-artemis pull request #2488: ARTEMIS-196 Implement Consumer Priority

2019-01-03 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2488#discussion_r245007330 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/QueueConsumersImpl.java --- @@ -0,0 +1,648 @@

[GitHub] activemq-artemis pull request #2488: ARTEMIS-196 Implement Consumer Priority

2019-01-03 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2488#discussion_r245007052 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/QueueConsumersImpl.java --- @@ -0,0 +1,648 @@

[GitHub] activemq-artemis pull request #2488: ARTEMIS-196 Implement Consumer Priority

2019-01-03 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2488#discussion_r245006670 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/QueueConsumersImpl.java --- @@ -0,0 +1,648 @@

[GitHub] activemq-artemis pull request #2488: ARTEMIS-196 Implement Consumer Priority

2019-01-03 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2488#discussion_r245006519 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/QueueConsumers.java --- @@ -0,0 +1,34 @@ +/*

[GitHub] activemq-artemis pull request #2488: ARTEMIS-196 Implement Consumer Priority

2019-01-03 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2488#discussion_r245006326 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/server/Consumer.java --- @@ -21,7 +21,7 @@ import

[GitHub] activemq-artemis pull request #2488: ARTEMIS-196 Implement Consumer Priority

2019-01-03 Thread franz1981
Github user franz1981 commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2488#discussion_r244987340 --- Diff: artemis-protocols/artemis-amqp-protocol/src/main/java/org/apache/activemq/artemis/protocol/amqp/broker/AMQPSessionCallback.java ---

[GitHub] activemq-artemis pull request #2488: ARTEMIS-196 Implement Consumer Priority

2019-01-03 Thread franz1981
Github user franz1981 commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2488#discussion_r244986539 --- Diff: artemis-core-client/src/main/java/org/apache/activemq/artemis/core/protocol/core/impl/wireformat/SessionCreateConsumerMessage.java ---

[GitHub] activemq-artemis pull request #2488: ARTEMIS-196 Implement Consumer Priority

2019-01-03 Thread franz1981
Github user franz1981 commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2488#discussion_r244986350 --- Diff: artemis-core-client/src/main/java/org/apache/activemq/artemis/core/protocol/core/impl/wireformat/SessionCreateConsumerMessage.java ---

[GitHub] activemq-artemis pull request #2488: ARTEMIS-196 Implement Consumer Priority

2019-01-03 Thread franz1981
Github user franz1981 commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2488#discussion_r244985399 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/ResetableIterator.java --- @@ -0,0 +1,29 @@ +/* +

[GitHub] activemq-artemis pull request #2488: ARTEMIS-196 Implement Consumer Priority

2019-01-03 Thread franz1981
Github user franz1981 commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2488#discussion_r244984864 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/QueueImpl.java --- @@ -3080,45 +3053,20 @@ private boolean

[GitHub] activemq-artemis pull request #2488: ARTEMIS-196 Implement Consumer Priority

2019-01-03 Thread franz1981
Github user franz1981 commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2488#discussion_r244984274 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/QueueImpl.java --- @@ -2497,45 +2494,36 @@ private void

[GitHub] activemq-artemis pull request #2488: ARTEMIS-196 Implement Consumer Priority

2019-01-03 Thread franz1981
Github user franz1981 commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2488#discussion_r244982460 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/QueueConsumersImpl.java --- @@ -0,0 +1,648 @@ +/*

[GitHub] activemq-artemis pull request #2488: ARTEMIS-196 Implement Consumer Priority

2019-01-03 Thread franz1981
Github user franz1981 commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2488#discussion_r244980096 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/QueueConsumersImpl.java --- @@ -0,0 +1,648 @@ +/*

[GitHub] activemq-artemis pull request #2488: ARTEMIS-196 Implement Consumer Priority

2019-01-03 Thread franz1981
Github user franz1981 commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2488#discussion_r244979227 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/QueueConsumersImpl.java --- @@ -0,0 +1,648 @@ +/*

[GitHub] activemq-artemis pull request #2488: ARTEMIS-196 Implement Consumer Priority

2019-01-03 Thread franz1981
Github user franz1981 commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2488#discussion_r244979032 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/QueueConsumersImpl.java --- @@ -0,0 +1,648 @@ +/*

[GitHub] activemq-artemis pull request #2488: ARTEMIS-196 Implement Consumer Priority

2019-01-03 Thread franz1981
Github user franz1981 commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2488#discussion_r244978399 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/QueueConsumers.java --- @@ -0,0 +1,34 @@ +/* + *

[GitHub] activemq-artemis pull request #2483: ARTEMIS-2215 largemessage have been con...

2019-01-03 Thread franz1981
Github user franz1981 commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2483#discussion_r244977454 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/persistence/impl/journal/JournalStorageManager.java --- @@ -727,7

[GitHub] activemq-artemis pull request #2488: ARTEMIS-196 Implement Consumer Priority

2019-01-03 Thread franz1981
Github user franz1981 commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2488#discussion_r244976790 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/server/Consumer.java --- @@ -21,7 +21,7 @@ import

[GitHub] activemq-artemis pull request #2483: ARTEMIS-2215 largemessage have been con...

2019-01-03 Thread CNNJYB
Github user CNNJYB commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2483#discussion_r244976446 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/persistence/impl/journal/JournalStorageManager.java --- @@ -309,16

[GitHub] activemq-artemis pull request #2488: ARTEMIS-196 Implement Consumer Priority

2019-01-03 Thread michaelandrepearce
GitHub user michaelandrepearce opened a pull request: https://github.com/apache/activemq-artemis/pull/2488 ARTEMIS-196 Implement Consumer Priority Add consumer priority support Includes refactor of consumer iterating in QueueImpl to its own logical class, to be able to

[GitHub] activemq-artemis pull request #2487: ARTEMIS-196 Implement Consumer Priority

2019-01-03 Thread michaelandrepearce
Github user michaelandrepearce closed the pull request at: https://github.com/apache/activemq-artemis/pull/2487 ---

[GitHub] activemq-artemis pull request #2487: ARTEMIS-196 Implement Consumer Priority

2019-01-03 Thread michaelandrepearce
GitHub user michaelandrepearce opened a pull request: https://github.com/apache/activemq-artemis/pull/2487 ARTEMIS-196 Implement Consumer Priority Add consumer priority support Includes refactor of consumer iterating in queueimpl to its own logical class, to be able to

[GitHub] activemq-artemis pull request #2486: ARTEMIS-196 Implement Consumer Priority

2019-01-03 Thread michaelandrepearce
Github user michaelandrepearce closed the pull request at: https://github.com/apache/activemq-artemis/pull/2486 ---

[GitHub] activemq-artemis pull request #2486: ARTEMIS-196 Implement Consumer Priority

2019-01-03 Thread michaelandrepearce
Github user michaelandrepearce closed the pull request at: https://github.com/apache/activemq-artemis/pull/2486 ---

[GitHub] activemq-artemis pull request #2486: ARTEMIS-196 Implement Consumer Priority

2019-01-03 Thread michaelandrepearce
GitHub user michaelandrepearce reopened a pull request: https://github.com/apache/activemq-artemis/pull/2486 ARTEMIS-196 Implement Consumer Priority Add consumer priority support Includes refactor of consumer iterating in queueimpl to its own logical class, to be able to

[GitHub] activemq-artemis pull request #2486: ARTEMIS-196 Implement Consumer Priority

2019-01-02 Thread michaelandrepearce
GitHub user michaelandrepearce opened a pull request: https://github.com/apache/activemq-artemis/pull/2486 ARTEMIS-196 Implement Consumer Priority Add consumer priority support Includes refactor of consumer iterating in queueimpl to its own logical class, to be able to

[GitHub] activemq-artemis pull request #2483: ARTEMIS-2215 largemessage have been con...

2019-01-02 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2483#discussion_r244757759 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/persistence/impl/journal/JournalStorageManager.java --- @@

[GitHub] activemq-artemis pull request #2483: ARTEMIS-2215 largemessage have been con...

2019-01-02 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2483#discussion_r244756852 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/persistence/impl/journal/JournalStorageManager.java --- @@

[GitHub] activemq-artemis pull request #2483: ARTEMIS-2215 largemessage have been con...

2019-01-02 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2483#discussion_r244686021 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/persistence/impl/journal/AbstractJournalStorageManager.java ---

[GitHub] activemq-artemis pull request #2483: ARTEMIS-2215 largemessage have been con...

2019-01-02 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2483#discussion_r244685607 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/persistence/impl/journal/JournalStorageManager.java --- @@

[GitHub] activemq-artemis pull request #2483: ARTEMIS-2215 largemessage have been con...

2019-01-01 Thread CNNJYB
Github user CNNJYB commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2483#discussion_r244657469 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/persistence/impl/journal/AbstractJournalStorageManager.java --- @@ -193,7

[GitHub] activemq-artemis pull request #2485: ARTEMIS-2217 ‘MQTTSessionState’ in ...

2018-12-29 Thread onlyMIT
GitHub user onlyMIT opened a pull request: https://github.com/apache/activemq-artemis/pull/2485 ARTEMIS-2217 ‘MQTTSessionState’ in the ‘SESSIONS ConcurrentHashMap’ n… …ever be removed ‘MQTTSessionState’ in the ‘SESSIONS ConcurrentHashMap’ should be

[GitHub] activemq-artemis pull request #2483: ARTEMIS-2215 largemessage have been con...

2018-12-29 Thread franz1981
Github user franz1981 commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2483#discussion_r244470771 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/persistence/impl/journal/JournalStorageManager.java --- @@ -461,8

[GitHub] activemq-artemis pull request #2483: ARTEMIS-2215 largemessage have been con...

2018-12-29 Thread franz1981
Github user franz1981 commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2483#discussion_r244470859 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/persistence/impl/journal/AbstractJournalStorageManager.java --- @@

[GitHub] activemq-artemis pull request #2484: Use a specific executor for pageSyncTim...

2018-12-29 Thread qihongxu
GitHub user qihongxu opened a pull request: https://github.com/apache/activemq-artemis/pull/2484 Use a specific executor for pageSyncTimer Improve paging throughput by using a specific executor for pageSyncTimer Improving throughput on paging mode is one of our concerns

[GitHub] activemq-artemis pull request #2483: ARTEMIS-2215 largemessage have been con...

2018-12-29 Thread CNNJYB
GitHub user CNNJYB opened a pull request: https://github.com/apache/activemq-artemis/pull/2483 ARTEMIS-2215 largemessage have been consumed but not deleted During the backup and live synchronization, the client consumes the largemessage, then the live crash(the

[GitHub] activemq-artemis pull request #2482: ARTEMIS-2214 Cache durable in ...

2018-12-27 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2482#discussion_r244150563 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/paging/cursor/PagedReferenceImpl.java --- @@ -74,6 +74,10 @@

[GitHub] activemq-artemis pull request #2482: ARTEMIS-2214 Cache durable in ...

2018-12-27 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2482#discussion_r244150155 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/paging/cursor/PagedReferenceImpl.java --- @@ -363,4 +371,20

[GitHub] activemq-artemis pull request #2482: ARTEMIS-2214 Cache durable in ...

2018-12-27 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2482#discussion_r244149886 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/paging/cursor/PagedReferenceImpl.java --- @@ -74,6 +74,10 @@

[GitHub] activemq-artemis pull request #2482: ARTEMIS-2214 Cache durable in ...

2018-12-25 Thread qihongxu
GitHub user qihongxu opened a pull request: https://github.com/apache/activemq-artemis/pull/2482 ARTEMIS-2214 Cache durable in PagedReference We recently performed a test on artemis broker and found a severe performance issue. When paged messages are being consumed,

[GitHub] activemq-artemis pull request #2481: ARTEMIS-2213 don't expire critical comp...

2018-12-24 Thread wy96f
GitHub user wy96f opened a pull request: https://github.com/apache/activemq-artemis/pull/2481 ARTEMIS-2213 don't expire critical component in the case of clock back drift In our production cluster some brokers crashed. There is nothing unusual in the dump stack. After digging into

[GitHub] activemq-artemis pull request #2478: ARTEMIS-2210 PagingStore creation is no...

2018-12-24 Thread gaohoward
Github user gaohoward commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2478#discussion_r243866712 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/paging/impl/PagingManagerImpl.java --- @@ -335,19 +335,25 @@ public

[GitHub] activemq-artemis pull request #2480: ARTEMIS-2212 Avoid using CLQ on ServerC...

2018-12-24 Thread franz1981
GitHub user franz1981 opened a pull request: https://github.com/apache/activemq-artemis/pull/2480 ARTEMIS-2212 Avoid using CLQ on ServerConsumerImpl It would deliver a better performance for the most common operations eg offer, poll, iterations, size. You can merge this pull

[GitHub] activemq-artemis pull request #2478: ARTEMIS-2210 PagingStore creation is no...

2018-12-24 Thread franz1981
Github user franz1981 commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2478#discussion_r243843391 --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/paging/impl/PagingManagerImpl.java --- @@ -335,19 +335,25 @@ public

[GitHub] activemq-artemis pull request #2479: ARTEMIS-2211 Avoid duplicate code for B...

2018-12-24 Thread franz1981
GitHub user franz1981 opened a pull request: https://github.com/apache/activemq-artemis/pull/2479 ARTEMIS-2211 Avoid duplicate code for ByteBuffer pooling and alignment Refactored thread local ByteBuffer pooling for both NIO & MAPPED seq file factories and used fast branchless

[GitHub] activemq-artemis pull request #:

2018-12-23 Thread franz1981
Github user franz1981 commented on the pull request: https://github.com/apache/activemq-artemis/commit/dfa70680fed37d25aa3a6d0d6a0795e580495b6a#commitcomment-31769915 No need to synchronize it: `stores` is a concurrent hashmap and you can use the atomic `computeIfAbsent` that use a

[GitHub] activemq-artemis pull request #2478: ARTEMIS-2210 PagingStore creation is no...

2018-12-23 Thread gaohoward
GitHub user gaohoward opened a pull request: https://github.com/apache/activemq-artemis/pull/2478 ARTEMIS-2210 PagingStore creation is not properly synchronized In PagingManagerImpl#getPageStore() the operations on the map 'stores' are not synchronzed and it's possible that more

[GitHub] activemq-artemis pull request #2477: ARTEMIS-2190 move tests

2018-12-21 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/2477 ARTEMIS-2190 move tests The "jms-tests" module is deprecated and these tests should have never gone in there. Moving them to the "integration-tests" module. You can merge this pull

[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 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 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 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

[GitHub] activemq-artemis pull request #2473: ARTEMIS-1058 Jars in web tmp dir locked...

2018-12-19 Thread gaohoward
Github user gaohoward commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2473#discussion_r243139995 --- Diff: artemis-web/src/main/java/org/apache/activemq/artemis/component/WebServerComponent.java --- @@ -237,28 +236,39 @@ public void

[GitHub] activemq-artemis pull request #2473: ARTEMIS-1058 Jars in web tmp dir locked...

2018-12-19 Thread clebertsuconic
Github user clebertsuconic commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2473#discussion_r243083234 --- Diff: artemis-web/src/main/java/org/apache/activemq/artemis/component/WebServerComponent.java --- @@ -237,28 +236,39 @@ public void

[GitHub] activemq-artemis pull request #2470: Fixes for alerts from lgtm.com

2018-12-19 Thread jbertram
Github user jbertram commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2470#discussion_r243076837 --- Diff: artemis-cli/src/main/java/org/apache/activemq/artemis/cli/commands/tools/PrintData.java --- @@ -216,63 +216,63 @@ private static void

[GitHub] activemq-artemis pull request #2470: Fixes for alerts from lgtm.com

2018-12-19 Thread clebertsuconic
Github user clebertsuconic commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2470#discussion_r243073035 --- Diff: artemis-cli/src/main/java/org/apache/activemq/artemis/cli/commands/tools/PrintData.java --- @@ -216,63 +216,63 @@ private

[GitHub] activemq-artemis pull request #2464: ARTEMIS-1859 Incorrect routing with AMQ...

2018-12-19 Thread jbertram
Github user jbertram commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2464#discussion_r243026861 --- Diff: artemis-protocols/artemis-amqp-protocol/src/main/java/org/apache/activemq/artemis/protocol/amqp/proton/ProtonServerReceiverContext.java

<    1   2   3   4   5   6   7   8   9   10   >