You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@activemq.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2017/08/09 13:08:00 UTC

[jira] [Commented] (ARTEMIS-1335) Update Netty to 4.1.14

    [ https://issues.apache.org/jira/browse/ARTEMIS-1335?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16119872#comment-16119872 ] 

ASF GitHub Bot commented on ARTEMIS-1335:
-----------------------------------------

Github user michaelandrepearce commented on a diff in the pull request:

    https://github.com/apache/activemq-artemis/pull/1452#discussion_r132177518
  
    --- Diff: artemis-journal/src/main/java/io/netty/buffer/UnpooledUnsafeDirectByteBufWrapper.java ---
    @@ -563,7 +564,11 @@ protected SwappedByteBuf newSwappedByteBuf() {
        @Override
        public ByteBuf setZero(int index, int length) {
           if (hasMemoryAddress()) {
    -         UnsafeByteBufUtil.setZero(this, addr(index), index, length);
    +         if (length == 0) {
    --- End diff --
    
    We have to do this as UnsafeByteBufUtil.setZero(this, addr(index), index, length); util method is removed in 4.1.14 this was the underlying util logic.



> Update Netty to 4.1.14
> ----------------------
>
>                 Key: ARTEMIS-1335
>                 URL: https://issues.apache.org/jira/browse/ARTEMIS-1335
>             Project: ActiveMQ Artemis
>          Issue Type: Improvement
>            Reporter: Michael Andre Pearce
>
> Netty just released a bunch of improvements and bug fixes in 4.1.14 especially around epoll native, which we should look to take advantage of.
> Also to note recent update of qpid-jms and jproton are dependent on 4.1.14 also.
> As such its prudent to update artemis dependency to 4.1.14.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)