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

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

I have the impression that this dependency is transitive due to the licensing 
issues of findbugs/spotbugs. This is not ideal (or user friendly) but I don't 
think we can have a straight workaround.

> javax.annotation.Nullable is missing for 
> org.apache.beam.sdk.schemas.FieldValueTypeInformation
> ----------------------------------------------------------------------------------------------
>
>                 Key: BEAM-8917
>                 URL: https://issues.apache.org/jira/browse/BEAM-8917
>             Project: Beam
>          Issue Type: Improvement
>          Components: sdk-java-core
>            Reporter: Tomo Suzuki
>            Assignee: Tomo Suzuki
>            Priority: Major
>          Time Spent: 40m
>  Remaining Estimate: 0h
>
> This ticket is from the result of static analysis by Linkage Checker 
> ([detail|https://github.com/GoogleCloudPlatform/cloud-opensource-java/issues/1045])
> h1. Example Project
> Example project to produce an issue: 
> https://github.com/suztomo/beam-java-sdk-missing-nullable .
> I think the Maven artifact {{org.apache.beam:beam-sdks-java-core}}, which 
> contains {{org.apache.beam.sdk.schemas.FieldValueTypeInformation}}, should 
> declare the dependency to {{com.google.code.findbugs:jsr305}}.
> h1. Why there's no problem in compilation and tests of sdks/java/core?
> The compilation succeeds because the {{Nullable}} annotation is in the 
> transitive dependency of compileOnly {{spotbugs-annotations}} dependency:
> {noformat}
> compileOnly - Compile only dependencies for source set 'main'.
> ...
> +--- com.github.spotbugs:spotbugs-annotations:3.1.12
> |    \--- com.google.code.findbugs:jsr305:3.0.2
> ...
> {noformat}
> The tests succeed because the {{Nullable}} annotation is in the transitive 
> dependency of {{guava-testlib}}.
> {noformat}
> testRuntime - Runtime dependencies for source set 'test' (deprecated, use 
> 'testRuntimeOnly' instead).
> ...
> +--- com.google.guava:guava-testlib:20.0
> |    +--- com.google.code.findbugs:jsr305:1.3.9
> {noformat}



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to