[GitHub] activemq-artemis issue #2015: ARTEMIS-1807 File-based Large Message encoding...

2018-04-21 Thread franz1981
Github user franz1981 commented on the issue: https://github.com/apache/activemq-artemis/pull/2015 That an histogram comparing 2.5 and this PR with 1 MB messages, default consumer window and a target throughput of 50 msg/sec: ![1m_50](https://user-images.githubusercontent.com/1312

[GitHub] activemq-artemis pull request #2015: ARTEMIS-1807 File-based Large Message e...

2018-04-21 Thread franz1981
Github user franz1981 closed the pull request at: https://github.com/apache/activemq-artemis/pull/2015 ---

[GitHub] activemq-artemis issue #2015: ARTEMIS-1807 File-based Large Message encoding...

2018-04-21 Thread franz1981
Github user franz1981 commented on the issue: https://github.com/apache/activemq-artemis/pull/2015 I?m clising this in the meantime proper tests will be performed :) ---

[GitHub] activemq-artemis pull request #2037: ARTEMIS-1822: Change the method name 'l...

2018-04-21 Thread BruceKuiLiu
GitHub user BruceKuiLiu opened a pull request: https://github.com/apache/activemq-artemis/pull/2037 ARTEMIS-1822: Change the method name 'lookupRecord' to 'containsRecord'. The method is named "lookupRecord". "lookupRecord" seems to find a related record. But the method is ch

[GitHub] activemq-artemis issue #2037: ARTEMIS-1822: Change the method name 'lookupRe...

2018-04-21 Thread jbertram
Github user jbertram commented on the issue: https://github.com/apache/activemq-artemis/pull/2037 The commit message should contain the JIRA, e.g. > ARTEMIS-1822 Change the method name 'lookupRecord' to 'containsRecord' Also, I noticed you've opened lots of similar PRs