[ 
https://issues.apache.org/jira/browse/BEAM-6770?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16785798#comment-16785798
 ] 

Ismaël Mejía commented on BEAM-6770:
------------------------------------

Yes it makes sense, if gradle does not suport optinal dependencies let's make 
it only runtime/testRuntime then and change the doc. Thanks [~michel] for the 
info.

> Correct zstd-jni dependency scope to optional
> ---------------------------------------------
>
>                 Key: BEAM-6770
>                 URL: https://issues.apache.org/jira/browse/BEAM-6770
>             Project: Beam
>          Issue Type: Improvement
>          Components: sdk-java-core
>            Reporter: Romain Manni-Bucau
>            Priority: Major
>
> 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).



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to