Details

    • Type: Sub-task Sub-task
    • Status: Patch Available
    • Priority: Minor Minor
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: 3.5.2, 3.6.0
    • Component/s: server
    • Labels:
      None

      Description

      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.

      1. ZOOKEEPER-910.patch
        1.0 kB
        Michi Mutsuzaki

        Activity

        No work has yet been logged on this issue.

          People

          • Assignee:
            Michi Mutsuzaki
            Reporter:
            Thomas Koch
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:

              Development