Accumulo
  1. Accumulo
  2. ACCUMULO-741

In SendLogToChainsaw, display CLI parameters upon parse error

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.5.0
    • Component/s: None
    • Labels:
      None

      Description

      The current error message is very terse "Missing required options: d, h, p, s, e". However, since we are using the Apache Commons CLI library it is easy t display a more comprehensive message.

        Activity

        Hide
        David Medinets added a comment -

        Changed server/src/main/java/org/apache/accumulo/server/util/SendLogToChainsaw.java to display:

        org.apache.commons.cli.MissingOptionException: Missing required options: d, h, p, s, e
        usage: SendLogToChainsaw
         -d,--logDirectory <dir>      ACCUMULO log directory path
         -e,--end <date>              end date filter (yyyyMMddHHmmss)
         -f,--fileFilter <filter>     filter to apply to names of logs
         -h,--host <hostname>         host where chainsaw is running
         -l,--level <level>           filter log level
         -m,--messageFilter <regex>   regex filter for log messages
         -p,--port <portnum>          port where XMLSocketReceiver is listening
         -s,--start <date>            start date filter (yyyyMMddHHmmss)
        
        Show
        David Medinets added a comment - Changed server/src/main/java/org/apache/accumulo/server/util/SendLogToChainsaw.java to display: org.apache.commons.cli.MissingOptionException: Missing required options: d, h, p, s, e usage: SendLogToChainsaw -d,--logDirectory <dir> ACCUMULO log directory path -e,--end <date> end date filter (yyyyMMddHHmmss) -f,--fileFilter <filter> filter to apply to names of logs -h,--host <hostname> host where chainsaw is running -l,--level <level> filter log level -m,--messageFilter <regex> regex filter for log messages -p,--port <portnum> port where XMLSocketReceiver is listening -s,--start <date> start date filter (yyyyMMddHHmmss)

          People

          • Assignee:
            David Medinets
            Reporter:
            David Medinets
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development