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 created issue -
          Harsh J made changes -
          Field Original Value New Value
          Attachment ZOOKEEPER-1326.patch [ 12508661 ]
          Harsh J made changes -
          Status Open [ 1 ] Patch Available [ 10002 ]
          Harsh J made changes -
          Status Patch Available [ 10002 ] Open [ 1 ]
          Harsh J made changes -
          Attachment ZOOKEEPER-1326.patch [ 12508663 ]
          Harsh J made changes -
          Status Open [ 1 ] Patch Available [ 10002 ]
          Patrick Hunt made changes -
          Assignee Harsh J [ qwertymaniac ]
          Patrick Hunt made changes -
          Fix Version/s 3.5.0 [ 12316644 ]
          Patrick Hunt made changes -
          Status Patch Available [ 10002 ] Resolved [ 5 ]
          Hadoop Flags Reviewed [ 10343 ]
          Resolution Fixed [ 1 ]
          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?
          Harsh J made changes -
          Link This issue supercedes ZOOKEEPER-729 [ ZOOKEEPER-729 ]

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Development