spotbugs-exclude.xml

<FindBugsFilter>
    <Match> <!-- does not work well with futures -->
        <Bug pattern="NP_NONNULL_PARAM_VIOLATION" />
    </Match>
    <Match> <!-- overagressive. Too many false positives -->
        <Bug pattern="RV_RETURN_VALUE_IGNORED_BAD_PRACTICE" />
    </Match>
    <Match> <!-- No practical consequence and generate a lot of warnings -->
        <Bug pattern="SS_SHOULD_BE_STATIC" />
    </Match>
    <Match> <!-- Usually done in a context where the developer knows the value to be positive -->
        <Bug pattern="IM_BAD_CHECK_FOR_ODD" />
    </Match>
    <Match> <!-- No practical consequence and generate a lot of warnings -->
        <Bug pattern="RV_RETURN_VALUE_IGNORED_NO_SIDE_EFFECT" />
    </Match>
    <Match> <!-- No practical consequence and generate a lot of warnings -->
        <Bug pattern="UC_USELESS_OBJECT" />
    </Match>
    <Match> <!-- No practical consequence and generate a lot of warnings -->
        <Bug pattern="DLS_DEAD_LOCAL_STORE" />
    </Match>
    <Match> <!-- Lots of harmless calls to System.exit() in our codebase -->
        <Bug pattern="DM_EXIT" />
    </Match>
    <Match> <!-- does not work from JDK11 onwards https://github.com/spotbugs/spotbugs/issues/756 -->
        <Bug pattern="RCN_REDUNDANT_NULLCHECK_WOULD_HAVE_BEEN_A_NPE" />
    </Match>
    <Match> <!-- does not work from JDK11 onwards https://github.com/spotbugs/spotbugs/issues/756 -->
        <Bug pattern="RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE" />
    </Match>
    <Match> <!-- does not work from JDK11 onwards https://github.com/spotbugs/spotbugs/issues/756 -->
        <Bug pattern="NP_NULL_ON_SOME_PATH_FROM_RETURN_VALUE" />
    </Match>

    <Match> <!-- Returning a java.lang.Boolean is common with Streams/Lambdas -->
        <Bug pattern="NP_BOOLEAN_RETURN_NULL" />
    </Match>
    <Match>
        <Bug pattern="ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD" />
    </Match>

    <Match>
        <Class name="io.pravega.sensor.collector.util.TransactionCoordinator" />
        <Bug pattern="CT_CONSTRUCTOR_THROW" />
    </Match>

    <Match>
        <Class name="io.pravega.sensor.collector.file.csvfile.CsvFileSequenceProcessor" />
        <Bug pattern="UR_UNINIT_READ_CALLED_FROM_SUPER_CONSTRUCTOR" />
    </Match>

    <Match>
        <Class name="io.pravega.sensor.collector.file.parquet.ParquetFileProcessor" />
        <Bug pattern="UR_UNINIT_READ_CALLED_FROM_SUPER_CONSTRUCTOR" />
    </Match>

    <Match>
        <Class name="io.pravega.sensor.collector.file.rawfile.RawFileProcessor" />
        <Bug pattern="UR_UNINIT_READ_CALLED_FROM_SUPER_CONSTRUCTOR" />
    </Match>

    <Match>
        <Class name="io.pravega.sensor.collector.util.TransactionCoordinatorTests" />
        <Bug pattern="ODR_OPEN_DATABASE_RESOURCE" />
    </Match>

    <Match>
        <Class name="io.pravega.sensor.collector.accelerometer.AccelerometerSamples" />
        <Bug pattern="PA_PUBLIC_PRIMITIVE_ATTRIBUTE" />
    </Match>

    <Match>
        <Class name="io.pravega.sensor.collector.network.NetworkSamples" />
        <Bug pattern="PA_PUBLIC_PRIMITIVE_ATTRIBUTE" />
    </Match>

    <Match> <!-- This is intended in these and only these classes -->
        <Or>
            <Bug pattern="EI_EXPOSE_REP" />
            <Bug pattern="EI_EXPOSE_REP2" />
        </Or>
    </Match>
    <Match> <!-- These patterns while normally bad practice, are common in tests -->
        <Class name="~.*\.*Tests?" />
        <Or>
            <Bug category="PERFORMANCE" />
            <Bug pattern="URF_UNREAD_PUBLIC_OR_PROTECTED_FIELD" />
            <Bug pattern="DMI_RANDOM_USED_ONLY_ONCE" />
        </Or>
    </Match>
</FindBugsFilter>