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

Brian Hulette commented on BEAM-10402:
--------------------------------------

Got it, thanks. It looks like the additional errors were places where checker 
framework could infer nullability, but adding the DefaultQualifier required us 
to make it explicit, e.g. 
https://github.com/apache/beam/blob/988b8042250f06f6a13cb5649a9d132f5df44514/sdks/java/extensions/zetasketch/src/main/java/org/apache/beam/sdk/extensions/zetasketch/HllCountMergePartialFn.java#L88

> [Umbrella] Eliminate nullability errors from Beam Java
> ------------------------------------------------------
>
>                 Key: BEAM-10402
>                 URL: https://issues.apache.org/jira/browse/BEAM-10402
>             Project: Beam
>          Issue Type: Improvement
>          Components: sdk-java-core
>            Reporter: Kenneth Knowles
>            Priority: P1
>              Labels: starter
>          Time Spent: 3h
>  Remaining Estimate: 0h
>
> I enabled the checker framework and the core SDK exceeds 100 nullability 
> errors immediately. We should make this number zero, since nullability errors 
> are solved problem in computer science and they do periodically impact Beam.



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

Reply via email to