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

removeWatches doesn't return NOWATCHER when there is no watch set

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 3.5.0
    • 3.5.0
    • server

    Description

      ZOOKEEPER-442 introduced support for a new opcode: removeWatches. The way it was implemented though, implies that you need to check on the client side if a watch/watcher is set before you send your request to the server. If you don't, ZK will just swallow your request and won't return an error code if there isn't a watch set for that path.

      I noticed this whilst implementing removeWatches for Kazoo [1]. As mentioned, I guess it could be expected that clients should do the check on their side but I think that the correct thing would to have the server do the validation and return the error code accordingly as well.

      rakeshr, phunt: thoughts?

      [1] https://github.com/rgs1/kazoo/commit/44ca48e975aeea3fd0664fe13136a72caf89e54f

      Attachments

        1. ZOOKEEPER-1909.patch
          10 kB
          Raúl Gutiérrez Segalés
        2. ZOOKEEPER-1909.patch
          10 kB
          Raúl Gutiérrez Segalés
        3. ZOOKEEPER-1909.patch
          10 kB
          Raúl Gutiérrez Segalés
        4. ZOOKEEPER-1909.patch
          4 kB
          Raúl Gutiérrez Segalés

        Activity

          People

            rgs Raúl Gutiérrez Segalés
            rgs Raúl Gutiérrez Segalés
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: