Accumulo
  1. Accumulo
  2. ACCUMULO-1899

cli.Help should return non-zero status on error parsing args

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.5.0
    • Fix Version/s: 1.5.1, 1.6.0
    • Component/s: None
    • Labels:

      Description

      Right now org.apache.accumulo.core.cli.Help.parseArgs() uses System.exit(0) on parse error.

      If the user gives invalid flags and triggers this code path, we should exit with non-zero status to indicate a problem.

      Previously (in 1.4.x), not properly setting non-zero status for command line parse errors caused some integration tests to fail silently for some time.

        Activity

        Sean Busbey created issue -
        Morgan Haskel made changes -
        Field Original Value New Value
        Status Open [ 1 ] Patch Available [ 10002 ]
        Morgan Haskel made changes -
        Attachment ACCUMULO-1899.patch [ 12615526 ]
        Josh Elser made changes -
        Assignee Morgan Haskel [ mhaskel ]
        Josh Elser made changes -
        Fix Version/s 1.5.1 [ 12324399 ]
        Fix Version/s 1.6.0 [ 12322468 ]
        Josh Elser made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Christopher Tubbs made changes -
        Affects Version/s 1.6.0 [ 12322468 ]

          People

          • Assignee:
            Morgan Haskel
            Reporter:
            Sean Busbey
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development