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

          Mike Drob created issue -
          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.
          Josh Elser made changes -
          Field Original Value New Value
          Fix Version/s 1.5.3 [ 12328662 ]
          Fix Version/s 1.5.2 [ 12326272 ]
          Corey J. Nolet made changes -
          Fix Version/s 1.6.2 [ 12328644 ]
          Fix Version/s 1.6.1 [ 12325441 ]
          Josh Elser made changes -
          Link This issue is duplicated by ACCUMULO-3279 [ ACCUMULO-3279 ]
          Josh Elser made changes -
          Assignee Josh Elser [ elserj ]
          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.
          Josh Elser made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Resolved Resolved
          190d 21h 21m 1 Josh Elser 31/Oct/14 17:47

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Development