[ANNOUNCE] Apache ActiveMQ 5.15.3 Released

2018-02-12 Thread Christopher Shannon
Hi everyone, The ActiveMQ team is pleased to announce that Apache ActiveMQ 5.15.3 has been released. A list of issues resolved in this release is available here: https://issues.apache.org/jira/secure/ReleaseNote.jspa? projectId=12311210=12341947 The Wiki page for the release is here:

[GitHub] activemq-artemis issue #1849: ARTEMIS-1656 OpenWire scalability improvements

2018-02-12 Thread franz1981
Github user franz1981 commented on the issue: https://github.com/apache/activemq-artemis/pull/1849 @michaelandrepearce > FYI gains were with non persistence Yep, I need to specify it on the JIRA thanks! I'm fixing the last few bits and it is getting in the right shape

[GitHub] activemq-artemis issue #1854: [ARTEMIS-1666] List of prepared transaction de...

2018-02-12 Thread yersan
Github user yersan commented on the issue: https://github.com/apache/activemq-artemis/pull/1854 @jbertram Note that there is no JMSServerControlTest test on master branch, but it exists on 1.x. This test can be used to cover this issue against regression, how do you usually deal with

[GitHub] activemq-artemis issue #1861: ARTEMIS-1675 Adding --safe option on print-dat...

2018-02-12 Thread jbertram
Github user jbertram commented on the issue: https://github.com/apache/activemq-artemis/pull/1861 Checkstyle violation in CoreMessage - extra space before the final semicolon. ---

[GitHub] activemq-artemis issue #1862: ARTEMIS-1676 Support overriding of JAVA_ARGS v...

2018-02-12 Thread jbertram
Github user jbertram commented on the issue: https://github.com/apache/activemq-artemis/pull/1862 I something similar possible for artemis.profile.cmd also? ---

[GitHub] activemq-artemis issue #1861: ARTEMIS-1675 Adding --safe option on print-dat...

2018-02-12 Thread clebertsuconic
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/1861 @jbertram i fixed while I merged it ---

[GitHub] activemq-artemis issue #1854: [ARTEMIS-1666] List of prepared transaction de...

2018-02-12 Thread jbertram
Github user jbertram commented on the issue: https://github.com/apache/activemq-artemis/pull/1854 If you don't want to do it let me know, and I'll do it. No worries. In any case, I think this PR should be closed. ---

[GitHub] activemq-artemis issue #1854: [ARTEMIS-1666] List of prepared transaction de...

2018-02-12 Thread yersan
Github user yersan commented on the issue: https://github.com/apache/activemq-artemis/pull/1854 @jbertram will it work for you if I create a new one PR on master branch and let this one open for 1.x? ---

[GitHub] activemq-artemis issue #1854: [ARTEMIS-1666] List of prepared transaction de...

2018-02-12 Thread jbertram
Github user jbertram commented on the issue: https://github.com/apache/activemq-artemis/pull/1854 Typically issues like this would be fixed on the 'master' branch first and then back-ported to any other branches to avoid regressions. Although the JMSServerManager is deprecated on

[GitHub] activemq-artemis issue #1854: [ARTEMIS-1666] List of prepared transaction de...

2018-02-12 Thread jbertram
Github user jbertram commented on the issue: https://github.com/apache/activemq-artemis/pull/1854 Ideally there would be a new PR on master and then a cherry-pick of that same commit onto 1.x (which would preclude this PR from staying open). ---

[GitHub] activemq-artemis issue #1854: [ARTEMIS-1666] List of prepared transaction de...

2018-02-12 Thread yersan
Github user yersan commented on the issue: https://github.com/apache/activemq-artemis/pull/1854 Ok, I'm going to close this one and create a new one for master and for 1.x (cherry-picked from master once it is merged) ---

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

2018-02-12 Thread jbertram
Github user jbertram commented on the issue: https://github.com/apache/activemq-artemis/pull/1850 Nice catch! ---

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

2018-02-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/activemq-artemis/pull/1850 ---

[GitHub] activemq-artemis pull request #1866: ARTEMIS-1660: Remove oracle12 autoincre...

2018-02-12 Thread graben
GitHub user graben opened a pull request: https://github.com/apache/activemq-artemis/pull/1866 ARTEMIS-1660: Remove oracle12 autoincrement from column id for journa… …l tables You can merge this pull request into a Git repository by running: $ git pull

[GitHub] activemq-artemis issue #1863: NO-JIRA: Sync OSGi broker config with latest c...

2018-02-12 Thread jbertram
Github user jbertram commented on the issue: https://github.com/apache/activemq-artemis/pull/1863 Is there no option to just copy (or build) this file from the source so manual syncing isn't necessary? ---

[GitHub] activemq-artemis pull request #1865: ARTEMIS-1666 refactor to avoid duplicat...

2018-02-12 Thread jbertram
GitHub user jbertram opened a pull request: https://github.com/apache/activemq-artemis/pull/1865 ARTEMIS-1666 refactor to avoid duplicate code and fix bug You can merge this pull request into a Git repository by running: $ git pull https://github.com/jbertram/activemq-artemis

[GitHub] activemq-artemis issue #1861: ARTEMIS-1675 Adding --safe option on print-dat...

2018-02-12 Thread clebertsuconic
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/1861 damn.. I think I had closed this... I will fix this. ---

[GitHub] activemq-artemis pull request #1854: [ARTEMIS-1666] List of prepared transac...

2018-02-12 Thread yersan
Github user yersan closed the pull request at: https://github.com/apache/activemq-artemis/pull/1854 ---

[GitHub] activemq-artemis pull request #1862: ARTEMIS-1676 Support overriding of JAVA...

2018-02-12 Thread mtaylor
Github user mtaylor commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1862#discussion_r167639675 --- Diff: artemis-cli/src/main/resources/org/apache/activemq/artemis/cli/commands/etc/artemis.profile --- @@ -21,14 +21,20 @@

[GitHub] activemq-artemis issue #1863: NO-JIRA: Sync OSGi broker config with latest c...

2018-02-12 Thread graben
Github user graben commented on the issue: https://github.com/apache/activemq-artemis/pull/1863 Well, actually the broker.xml is build from several templates and snippets. OSGi needs a complete file to trigger broker deployment. Therefore a a file need to be constructed during build

[GitHub] activemq-artemis issue #1854: [ARTEMIS-1666] List of prepared transaction de...

2018-02-12 Thread jbertram
Github user jbertram commented on the issue: https://github.com/apache/activemq-artemis/pull/1854 Although this has been merged the PR won't be closed automatically (something funny going on in the automation). @yersan, please close this PR manually. ---

[GitHub] activemq-artemis issue #1854: [ARTEMIS-1666] List of prepared transaction de...

2018-02-12 Thread yersan
Github user yersan commented on the issue: https://github.com/apache/activemq-artemis/pull/1854 @jbertram Done, thanks! ---

[GitHub] activemq-artemis issue #1862: ARTEMIS-1676 Support overriding of JAVA_ARGS v...

2018-02-12 Thread mtaylor
Github user mtaylor commented on the issue: https://github.com/apache/activemq-artemis/pull/1862 Yes it's possible. We would need to test it on a windows machine. That profile is actually structured differently ---

[GitHub] activemq-artemis issue #1854: [ARTEMIS-1666] List of prepared transaction de...

2018-02-12 Thread jbertram
Github user jbertram commented on the issue: https://github.com/apache/activemq-artemis/pull/1854 So forget what I said before. I'll merge this PR and then send a different one upstream to deal with this issue in a different way given the management objects have changed so

[GitHub] activemq-artemis pull request #1861: ARTEMIS-1675 Adding --safe option on pr...

2018-02-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/activemq-artemis/pull/1861 ---

[GitHub] activemq-artemis pull request #1863: NO-JIRA: Sync OSGi broker config with l...

2018-02-12 Thread graben
Github user graben commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1863#discussion_r167488599 --- Diff: artemis-features/src/main/resources/artemis.xml --- @@ -32,21 +32,23 @@ under the License. MAPPED: mmap files

[GitHub] activemq-artemis pull request #1863: NO-JIRA: Sync OSGi broker config with l...

2018-02-12 Thread graben
Github user graben commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1863#discussion_r167488412 --- Diff: artemis-features/src/main/resources/artemis.xml --- @@ -32,21 +32,23 @@ under the License. MAPPED: mmap files

[GitHub] activemq-artemis pull request #1863: NO-JIRA: Sync OSGi broker config with l...

2018-02-12 Thread graben
Github user graben commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1863#discussion_r167488780 --- Diff: artemis-features/src/main/resources/artemis.xml --- @@ -196,5 +194,20 @@ under the License. + +

[GitHub] activemq-artemis pull request #1867: ARTEMIS-1679 Stopping consumer no need ...

2018-02-12 Thread wy96f
GitHub user wy96f opened a pull request: https://github.com/apache/activemq-artemis/pull/1867 ARTEMIS-1679 Stopping consumer no need to wait until all queued tasks… Sping's SimpleMessageListenerContainer shutdown will stop consumers and then close them. It will take lots of