ZooKeeper
  1. ZooKeeper
  2. ZOOKEEPER-1326

The CLI commands "delete" and "rmr" are confusing. Can we have "delete" + "deleteall" instead?

    Details

    • Type: Wish Wish
    • Status: Resolved
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: 3.4.0
    • Fix Version/s: 3.5.0
    • Component/s: java client
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      ZOOKEEPER-729 introduced 'rmr' for recursive 'delete' operations on a given node. Going by the unix convention, wouldn't it be much better if we were to have an 'rm' if there was an 'rmr' added?

      The current set is confusing. Or should we have 'delete' and 'deleteall' or summat?

      I know this is a nitpick, but I just dislike to see bad keywords used for commands.

      I'm OK to produce a backwards-compatible patch if this is acceptable.

      1. ZOOKEEPER-1326.patch
        3 kB
        Harsh J
      2. ZOOKEEPER-1326.patch
        3 kB
        Harsh J

        Issue Links

          Activity

          Harsh J made changes -
          Link This issue supercedes ZOOKEEPER-729 [ ZOOKEEPER-729 ]
          Harsh J made changes -
          Summary The CLI commands "delete" and "rmr" are confusing. Can we have "rm" + "rmr" instead? The CLI commands "delete" and "rmr" are confusing. Can we have "delete" + "deleteall" instead?
          Patrick Hunt made changes -
          Status Patch Available [ 10002 ] Resolved [ 5 ]
          Hadoop Flags Reviewed [ 10343 ]
          Resolution Fixed [ 1 ]
          Patrick Hunt made changes -
          Fix Version/s 3.5.0 [ 12316644 ]
          Patrick Hunt made changes -
          Assignee Harsh J [ qwertymaniac ]
          Harsh J made changes -
          Status Open [ 1 ] Patch Available [ 10002 ]
          Harsh J made changes -
          Attachment ZOOKEEPER-1326.patch [ 12508663 ]
          Harsh J made changes -
          Status Patch Available [ 10002 ] Open [ 1 ]
          Harsh J made changes -
          Status Open [ 1 ] Patch Available [ 10002 ]
          Harsh J made changes -
          Field Original Value New Value
          Attachment ZOOKEEPER-1326.patch [ 12508661 ]
          Harsh J created issue -

            People

            • Assignee:
              Harsh J
              Reporter:
              Harsh J
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development