Accumulo
  1. Accumulo
  2. ACCUMULO-2727

version argument of ZooUtil.recursiveDelete is ignored

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.5.1, 1.6.0
    • Fix Version/s: 1.5.3, 1.6.2, 1.7.0
    • Component/s: fate
    • Labels:
      None

      Description

      The version argument of recursiveDelete is ignored. We should either use it, or remove it, instead of giving the illusion that we respect it.

        Issue Links

          Activity

          Hide
          Josh Elser added a comment -

          Gah, put the duplicate issue's ID instead of this one

              Commit 69d665cb4c387afd64f5f0b56e6f52eec2b188a4 in accumulo's branch refs/heads/1.5 from [~elserj]
              [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=69d665c ]
          
              ACCUMULO-3279 Remove version argument from recursiveDelete
          
              `recursiveDelete` is much like a big hammer: get rid of
              all of the nodes beneath this parent. As such, it doesn't
              make sense to pass in a version, nor does it make sense to
              provide a specific version to delete. We can pass in a version
              of '-1' which should eliminate the need for any special handling
              around BadVersionExceptions.
          
          Show
          Josh Elser added a comment - Gah, put the duplicate issue's ID instead of this one Commit 69d665cb4c387afd64f5f0b56e6f52eec2b188a4 in accumulo's branch refs/heads/1.5 from [~elserj] [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=69d665c ] ACCUMULO-3279 Remove version argument from recursiveDelete `recursiveDelete` is much like a big hammer: get rid of all of the nodes beneath this parent. As such, it doesn't make sense to pass in a version, nor does it make sense to provide a specific version to delete. We can pass in a version of '-1' which should eliminate the need for any special handling around BadVersionExceptions.
          Hide
          John Vines added a comment -

          ZK 3.4 added a recursive delete, which we could just use instead. We'd just have to wrap it to handle the NodeMIssingPOlicy support.

          Show
          John Vines added a comment - ZK 3.4 added a recursive delete, which we could just use instead. We'd just have to wrap it to handle the NodeMIssingPOlicy support.

            People

            • Assignee:
              Josh Elser
              Reporter:
              Mike Drob
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development