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

Correct zstd-jni dependency scope to test only

Details

    • Improvement
    • Status: Resolved
    • P2
    • Resolution: Fixed
    • None
    • 2.12.0
    • sdk-java-core

    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

              jeff.klukas@gmail.com Jeff Klukas
              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