zookeeper-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Andor Molnar (JIRA)" <j...@apache.org>
Subject [jira] [Assigned] (ZOOKEEPER-3457) Code optimization in QuorumCnxManager
Date Sat, 27 Jul 2019 18:06:00 GMT

     [ https://issues.apache.org/jira/browse/ZOOKEEPER-3457?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Andor Molnar reassigned ZOOKEEPER-3457:
---------------------------------------

    Assignee: tom.long

> Code optimization in QuorumCnxManager
> -------------------------------------
>
>                 Key: ZOOKEEPER-3457
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-3457
>             Project: ZooKeeper
>          Issue Type: Improvement
>          Components: quorum
>    Affects Versions: 3.5.5
>            Reporter: tom.long
>            Assignee: tom.long
>            Priority: Trivial
>              Labels: easyfix, pull-request-available
>             Fix For: 3.6.0
>
>   Original Estimate: 1h
>  Remaining Estimate: 1h
>
> Dear developer:
> I think the following code in line 623 of the QuorumCnxManager class can be optimized:
> {code:java}
> ArrayBlockingQueue<ByteBuffer> bq = new ArrayBlockingQueue<ByteBuffer>(
> SEND_CAPACITY);
> ArrayBlockingQueue<ByteBuffer> oldq = queueSendMap.putIfAbsent(sid, bq);
> if (oldq != null) {
> addToSendQueue(oldq, b);
> } else {
> addToSendQueue(bq, b);
> }
> {code}
> The optimization is as follows:
> {code:java}
> ArrayBlockingQueue<ByteBuffer> bq = queueSendMap.computeIfAbsent(sid, serverId

>  -> new ArrayBlockingQueue<>(SEND_CAPACITY));
> addToSendQueue(bq, b);
> {code}



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)

Mime
View raw message