Details

    • Type: Sub-task
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 3.0.0
    • Component/s: None
    • Labels:
      None

      Description

      ZooKeeper 3.5.0 added a new feature "Watcher Removal". Curator needs to support this. Further, all Curator recipe uses of watchers should clear their watchers on close, etc. as appropriate.

        Issue Links

          Activity

          Hide
          cammckenzie Cameron McKenzie added a comment -

          Jordan Zimmerman, are you looking at this currently? I was going to start having a look if you weren't.
          cheers

          Show
          cammckenzie Cameron McKenzie added a comment - Jordan Zimmerman , are you looking at this currently? I was going to start having a look if you weren't. cheers
          Hide
          randgalt Jordan Zimmerman added a comment -

          Go ahead

          Show
          randgalt Jordan Zimmerman added a comment - Go ahead
          Hide
          mdrob Mike Drob added a comment - - edited

          Build fails due to missing licenses:

          [INFO] Missing header in: curator-framework/src/main/java/org/apache/curator/framework/api/BackgroundPathableQuietlyable.java
          
          Show
          mdrob Mike Drob added a comment - - edited Build fails due to missing licenses: [INFO] Missing header in: curator-framework/src/main/java/org/apache/curator/framework/api/BackgroundPathableQuietlyable.java
          Hide
          randgalt Jordan Zimmerman added a comment -

          You can run:

          mvn license:format

          to add the license. I'll do it when I get a chance or you can.

          Show
          randgalt Jordan Zimmerman added a comment - You can run: mvn license:format to add the license. I'll do it when I get a chance or you can.
          Hide
          mdrob Mike Drob added a comment -

          Oh, I didn't know we had that. Ran it and pushed the changes.

          Show
          mdrob Mike Drob added a comment - Oh, I didn't know we had that. Ran it and pushed the changes.

            People

            • Assignee:
              cammckenzie Cameron McKenzie
              Reporter:
              randgalt Jordan Zimmerman
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development