This closes #4021: [BEAM-3081] Findbugs: NonNull by default NonNull by default in metrics Ignore findbugs in AutoValue generated classes NonNull by default for sdk/testing NonNull by default in sdk/state NonNull by default in sdk/runners NonNull by default in sdk/annotations NonNull by default in sdk/coders Make Java core SDK root dir NonNull by default Add dep on Apache-licensed findbugs-annotations implementation
Project: http://git-wip-us.apache.org/repos/asf/beam/repo Commit: http://git-wip-us.apache.org/repos/asf/beam/commit/66fcda99 Tree: http://git-wip-us.apache.org/repos/asf/beam/tree/66fcda99 Diff: http://git-wip-us.apache.org/repos/asf/beam/diff/66fcda99 Branch: refs/heads/master Commit: 66fcda99c1e5902ff4d0981b887c70f7afe9380a Parents: a29e0ad a1311d4 Author: Kenneth Knowles <[email protected]> Authored: Fri Oct 20 14:51:56 2017 -0700 Committer: Kenneth Knowles <[email protected]> Committed: Fri Oct 20 14:51:56 2017 -0700 ---------------------------------------------------------------------- pom.xml | 7 ++++ .../beam/runners/core/SideInputHandler.java | 2 +- .../apache/beam/runners/core/WatermarkHold.java | 6 +-- .../triggers/TriggerStateMachineRunner.java | 3 +- .../src/main/resources/beam/findbugs-filter.xml | 7 +++- sdks/java/core/pom.xml | 5 +++ .../main/java/org/apache/beam/sdk/Pipeline.java | 12 +++++- .../beam/sdk/annotations/package-info.java | 4 ++ .../apache/beam/sdk/coders/DefaultCoder.java | 17 ++++---- .../beam/sdk/coders/LengthPrefixCoder.java | 3 +- .../beam/sdk/coders/SerializableCoder.java | 5 ++- .../apache/beam/sdk/coders/StructuredCoder.java | 8 +--- .../org/apache/beam/sdk/coders/VoidCoder.java | 2 + .../apache/beam/sdk/coders/package-info.java | 4 ++ .../apache/beam/sdk/metrics/package-info.java | 4 ++ .../java/org/apache/beam/sdk/package-info.java | 4 ++ .../beam/sdk/runners/TransformHierarchy.java | 15 ++++--- .../apache/beam/sdk/runners/package-info.java | 4 ++ .../org/apache/beam/sdk/state/BagState.java | 6 +++ .../apache/beam/sdk/state/CombiningState.java | 5 +++ .../apache/beam/sdk/state/ReadableState.java | 4 +- .../org/apache/beam/sdk/state/StateSpecs.java | 4 +- .../org/apache/beam/sdk/state/package-info.java | 4 ++ .../beam/sdk/testing/FileChecksumMatcher.java | 41 +++++++++++++------- .../beam/sdk/testing/SuccessOrFailure.java | 6 +-- .../apache/beam/sdk/testing/TestPipeline.java | 4 +- .../apache/beam/sdk/testing/WindowSupplier.java | 4 +- .../apache/beam/sdk/testing/package-info.java | 5 +++ 28 files changed, 142 insertions(+), 53 deletions(-) ----------------------------------------------------------------------
