Uploaded image for project: 'ZooKeeper'
  1. ZooKeeper
  2. ZOOKEEPER-794

Callbacks are not invoked when the client is closed

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Blocker
    • Resolution: Fixed
    • 3.3.1
    • 3.3.2, 3.4.0
    • java client
    • None
    • Reviewed

    Description

      I noticed that ZooKeeper has different behaviors when calling synchronous or asynchronous actions on a closed ZooKeeper client.
      Actually a synchronous call will throw a "session expired" exception while an asynchronous call will do nothing. No exception, no callback invocation.

      Actually, even if the EventThread receives the Packet with the session expired err code, the packet is never processed since the thread has been killed by the ventOfDeath. So the call back is not invoked.

      Attachments

        1. ZOOKEEPER-794_2.patch
          15 kB
          Benjamin Reed
        2. ZOOKEEPER-794_3.patch
          5 kB
          Alexis Midon
        3. ZOOKEEPER-794_4.patch.txt
          16 kB
          Alexis Midon
        4. ZOOKEEPER-794_5_br33.patch
          17 kB
          Patrick D. Hunt
        5. ZOOKEEPER-794_5.patch.txt
          16 kB
          Alexis Midon
        6. ZOOKEEPER-794.patch.txt
          13 kB
          Alexis Midon
        7. ZOOKEEPER-794.txt
          13 kB
          Alexis Midon

        Issue Links

          is related to

          Bug - A problem which impairs or prevents the functions of the product. ZOOKEEPER-954 Findbugs/ClientCnxn: Bug type JLM_JSR166_UTILCONCURRENT_MONITORENTER

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Patch Available

          Improvement - An improvement or enhancement to an existing feature or task. ZOOKEEPER-835 Refactoring Zookeeper Client Code

          • Major - Major loss of function.
          • Open

          Activity

            People

              alexismidon Alexis Midon
              alexismidon Alexis Midon
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: