ZooKeeper
  1. ZooKeeper
  2. ZOOKEEPER-670

zkpython leading to segfault on zookeeper server restart

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: 3.2.1, 3.2.2
    • Fix Version/s: None
    • Component/s: contrib-bindings
    • Labels:
      None
    • Environment:

      CentOS w/ Python 2.4

      Description

      Zookeeper client using zkpython segfaults on zookeeper server restart. It is reliably reproducible using the attached script zk.py.
      I'm able to stop segfault using the attached patch voyager.patch, but zkpython seems to have deeper issue on its use of watcher_dispatch - on zookeeper server restart, I see up to 6 invocation of watcher_dispatch while my script is simply sleeping in the main thread. This can't be right.

      1. zk.py
        1 kB
        Lei Zhang
      2. voyager.patch
        1 kB
        Lei Zhang

        Issue Links

          Activity

          Hide
          Michi Mutsuzaki added a comment -

          This issue has been fixed in 3.4 by ZOOKEEPER-888.

          Show
          Michi Mutsuzaki added a comment - This issue has been fixed in 3.4 by ZOOKEEPER-888 .
          Hide
          Mahadev konar added a comment -

          not a blocker. Moving it out of 3.4 release.

          Show
          Mahadev konar added a comment - not a blocker. Moving it out of 3.4 release.
          Hide
          Andrei Savu added a comment -

          It seems like this is no longer an issue on the trunk (fixed by ZOOKEEPER-888). I've run zk.py against the latest version and I see no segfault but unfortunately on the 3.3 branch it still crashes.

          Show
          Andrei Savu added a comment - It seems like this is no longer an issue on the trunk (fixed by ZOOKEEPER-888 ). I've run zk.py against the latest version and I see no segfault but unfortunately on the 3.3 branch it still crashes.
          Hide
          Patrick Hunt added a comment -

          Not a blocker, pushing to 3.4.0

          Show
          Patrick Hunt added a comment - Not a blocker, pushing to 3.4.0
          Hide
          Patrick Hunt added a comment -

          Henry can you take a look at this one?

          Show
          Patrick Hunt added a comment - Henry can you take a look at this one?

            People

            • Assignee:
              Henry Robinson
              Reporter:
              Lei Zhang
            • Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development