Uploaded image for project: 'ZooKeeper'
  1. ZooKeeper
  2. ZOOKEEPER-1198 Refactorings and Cleanups
  3. ZOOKEEPER-1217

Remove unnecessary MissingSessionException in ZooKeeperServer

    XMLWordPrintableJSON

    Details

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

      Description

      MissingSessionException in only thrown and catched once inside this class and can as well be replaced by a boolean return value.

      While I'm at it: The method throwing this Exception makes more sense to be inlined in the one place from where it is called.

        Attachments

        1. ZOOKEEPER-1217.patch
          4 kB
          Thomas Koch
        2. ZOOKEEPER-1217.patch
          4 kB
          Thomas Koch
        3. ZOOKEEPER-1217.patch
          4 kB
          Thomas Koch
        4. ZOOKEEPER-1217.patch
          4 kB
          Thomas Koch
        5. ZOOKEEPER-1217.patch
          4 kB
          Thomas Koch

          Activity

            People

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

              Dates

              • Created:
                Updated: