[GitHub] activemq-artemis pull request #1786: ARTEMIS-1616 OpenWire improvements

2018-01-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/activemq-artemis/pull/1786 ---

[GitHub] activemq-artemis pull request #1786: ARTEMIS-1616 OpenWire improvements

2018-01-22 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1786#discussion_r163012542 --- Diff: artemis-commons/src/main/java/org/apache/activemq/artemis/api/core/SimpleString.java --- @@ -361,13 +371,68 @@ public int

[GitHub] activemq-artemis pull request #1786: ARTEMIS-1616 OpenWire improvements

2018-01-19 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1786#discussion_r162637206 --- Diff: artemis-commons/src/main/java/org/apache/activemq/artemis/api/core/SimpleString.java --- @@ -361,13 +371,68 @@ public int

[GitHub] activemq-artemis pull request #1786: ARTEMIS-1616 OpenWire improvements

2018-01-19 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1786#discussion_r162636748 --- Diff: artemis-commons/src/main/java/org/apache/activemq/artemis/api/core/SimpleString.java --- @@ -361,13 +371,68 @@ public int

[GitHub] activemq-artemis pull request #1786: ARTEMIS-1616 OpenWire improvements

2018-01-19 Thread franz1981
Github user franz1981 commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1786#discussion_r162610245 --- Diff: artemis-commons/src/main/java/org/apache/activemq/artemis/api/core/SimpleString.java --- @@ -361,13 +371,68 @@ public int hashCode()

[GitHub] activemq-artemis pull request #1786: ARTEMIS-1616 OpenWire improvements

2018-01-18 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1786#discussion_r162492929 --- Diff: artemis-commons/src/main/java/org/apache/activemq/artemis/api/core/SimpleString.java --- @@ -361,13 +371,68 @@ public int

[GitHub] activemq-artemis pull request #1786: ARTEMIS-1616 OpenWire improvements

2018-01-18 Thread franz1981
Github user franz1981 commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1786#discussion_r162272711 --- Diff: artemis-commons/src/main/java/org/apache/activemq/artemis/api/core/SimpleString.java --- @@ -361,13 +371,68 @@ public int hashCode()

[GitHub] activemq-artemis pull request #1786: ARTEMIS-1616 OpenWire improvements

2018-01-18 Thread franz1981
Github user franz1981 commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1786#discussion_r162272578 --- Diff: artemis-commons/src/main/java/org/apache/activemq/artemis/api/core/SimpleString.java --- @@ -361,13 +371,68 @@ public int hashCode()

[GitHub] activemq-artemis pull request #1786: ARTEMIS-1616 OpenWire improvements

2018-01-17 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1786#discussion_r162266546 --- Diff: artemis-commons/src/main/java/org/apache/activemq/artemis/api/core/SimpleString.java --- @@ -361,13 +371,68 @@ public int

[GitHub] activemq-artemis pull request #1786: ARTEMIS-1616 OpenWire improvements

2018-01-17 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1786#discussion_r162262699 --- Diff: artemis-commons/src/main/java/org/apache/activemq/artemis/api/core/SimpleString.java --- @@ -361,13 +371,68 @@ public int

[GitHub] activemq-artemis pull request #1786: ARTEMIS-1616 OpenWire improvements

2018-01-17 Thread michaelandrepearce
Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1786#discussion_r162262393 --- Diff: artemis-commons/src/main/java/org/apache/activemq/artemis/api/core/SimpleString.java --- @@ -361,13 +371,68 @@ public int

[GitHub] activemq-artemis pull request #1786: ARTEMIS-1616 OpenWire improvements

2018-01-17 Thread franz1981
GitHub user franz1981 opened a pull request: https://github.com/apache/activemq-artemis/pull/1786 ARTEMIS-1616 OpenWire improvements It includes several improvements on OpenWire: - Avoided copy of CoreMessage when not needed and cached lambda on hot path - Refactored