Uploaded image for project: 'Beam'
  1. Beam
  2. BEAM-6770

Correct zstd-jni dependency scope to test only

    XMLWordPrintableJSON

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.12.0
    • Component/s: sdk-java-core
    • Labels:
      None

      Description

      Beam 2.11.0 introduced a new transitive dep aka zstd-jni. AFAIK it is not needed in most cases so shouldn't be here by default. Also saw it was configured as shadow in the sdk core java module so not sure it is a gradle build bug or intended to be like that but I think sdk-core-java should be cleaned up cause it is now very fast and does not match a lot of usage. Finally this lib being native it is not that sane to bring it by default, in particular with the dockerization happening right now and the goal to have a light container stack (which often implies to not use standard linux as FROM).

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                jeff.klukas@gmail.com Jeff Klukas
                Reporter:
                romain.manni-bucau Romain Manni-Bucau
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0h
                  0h
                  Logged:
                  Time Spent - 0.5h
                  0.5h