[
https://issues.apache.org/jira/browse/ZOOKEEPER-910?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Zili Chen reassigned ZOOKEEPER-910:
-----------------------------------
Assignee: Zili Chen (was: Michi Mutsuzaki)
> Use SelectionKey.isXYZ() methods instead of complicated binary logic
> --------------------------------------------------------------------
>
> Key: ZOOKEEPER-910
> URL: https://issues.apache.org/jira/browse/ZOOKEEPER-910
> Project: ZooKeeper
> Issue Type: Sub-task
> Components: server
> Reporter: Thomas Koch
> Assignee: Zili Chen
> Priority: Minor
> Labels: pull-request-available
> Attachments: ZOOKEEPER-910.patch
>
> Time Spent: 10m
> Remaining Estimate: 0h
>
> The SelectionKey class provides methods to replace something like this
> (k.readyOps() & (SelectionKey.OP_READ | SelectionKey.OP_WRITE)) != 0
> with
> selectionKey.isReadable() || selectionKey.isWritable()
> It may be possible, that the first version saves a CPU cycle or two, but the
> later version saves developer brain cycles which are much more expensive.
> I suppose that there are many more places in the server code where this
> replacement could be done. I propose that whoever touches a code line like
> this should make the replacement.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
