Uploaded image for project: 'ZooKeeper'
  1. ZooKeeper
  2. ZOOKEEPER-271

Better command line parsing in ZookeeperMain.

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 3.0.0, 3.0.1
    • Fix Version/s: 3.5.0
    • Component/s: java client
    • Labels:
      None
    • Hadoop Flags:
      Incompatible change, Reviewed
    • Release Note:
      Hide
      There are three incompatibilities introduced by this commit into the client shell:

      1) now requires commons-cli
      2) get no longer returns stat information by default, however there is a "-s" option that will result in the stat being included
      3) a deprecated message is reported in some cases, when the old command format is used. As a result the output of the command may be different compared to client output prior to this change.
      Show
      There are three incompatibilities introduced by this commit into the client shell: 1) now requires commons-cli 2) get no longer returns stat information by default, however there is a "-s" option that will result in the stat being included 3) a deprecated message is reported in some cases, when the old command format is used. As a result the output of the command may be different compared to client output prior to this change.

      Description

      The command line parsing in zookeepermain is very basic.We should use some kind of cli parsing (commons-cli?) or something else that is standard and improve our command line parsing. This will remove the scattered code that we have in zookeepermain and we will have much better command line parsing.

        Attachments

        1. ZOOKEEPER-271.patch
          77 kB
          Hartmut Lang
        2. ZOOKEEPER-271-1.patch
          77 kB
          Hartmut Lang

          Issue Links

            Activity

              People

              • Assignee:
                hlang Hartmut Lang
                Reporter:
                mahadev Mahadev konar
              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: