Uploaded image for project: 'Phoenix'
  1. Phoenix
  2. PHOENIX-6349

Add and use commons-cli to phoenix-thirdparty

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 5.1.0, 4.17.0, thirdparty-1.1.0
    • None
    • None
    • Hide
      Added phoenix-shaded-commons-cli to phoenix-thirdparty.
      This helps avoid classpath conflicts and includes a patch on top of commons-cli 1.4.0 that fixes CLI-254.
      Show
      Added phoenix-shaded-commons-cli to phoenix-thirdparty. This helps avoid classpath conflicts and includes a patch on top of commons-cli 1.4.0 that fixes CLI-254 .

    Description

      We are hit hard by commons-cli bugs, and currently we msut stick to the version used by HBase/Hadoop.

      Adding commons-cli to phoenix-thirdparty would unchain us from the Hbase/Hadoop version, and would let us add our own patches (modeled after protobuf in HBase)

      Attachments

        Issue Links

          is depended upon by

          Improvement - An improvement or enhancement to an existing feature or task. PHOENIX-6357 Change all command line tools to use the fixed commons-cli constructor

          • Major - Major loss of function.
          • Resolved
          relates to

          Bug - A problem which impairs or prevents the functions of the product. PHOENIX-6348 java.lang.NoClassDefFoundError: when running with hbase-1.6

          • Blocker - Blocks development and/or testing work, production could not run
          • Resolved

          Bug - A problem which impairs or prevents the functions of the product. CLI-254 "test" gets parsed as test, quotes die :-(

          • Major - Major loss of function.
          • Resolved
          links to

          Web Link GitHub Pull Request #2

          Web Link GitHub Pull Request #1122

          Web Link GitHub Pull Request #1123

          Activity

            People

              stoty Istvan Toth
              stoty Istvan Toth
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: