Uploaded image for project: 'ZooKeeper'
  1. ZooKeeper
  2. ZOOKEEPER-835 Refactoring Zookeeper Client Code
  3. ZOOKEEPER-839

deleteRecursive does not belong to the other methods

    XMLWordPrintableJSON

    Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Blocker
    • Resolution: Fixed
    • Affects Version/s: 3.3.1
    • Fix Version/s: 3.4.0
    • Component/s: java client
    • Labels:
      None
    • Hadoop Flags:
      Reviewed
    • Tags:
      atomic operations

      Description

      DeleteRecursive has been committed to trunk already as a method to the
      zookeeper class. So in the API it has the same level as the atomic operations
      create, delete, getData, setData, etc. The user must get the false impression,
      that deleteRecursive is also an atomic operation.
      It would be better to have deleteRecursive in some helper class but not that
      deep in zookeeper's core code. Maybe I'd like to have another policy on how to
      react if deleteRecursive fails in the middle of its work?

        Attachments

        1. ZOOKEEPER-839.patch
          11 kB
          Mahadev Konar

          Activity

            People

            • Assignee:
              mahadev Mahadev Konar
              Reporter:
              ymc-pada Patrick Datko
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: