[KAFKA-4773] The Kafka build should run findbugs

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 0.11.0.0
• Assignee: Colin McCabe
• Reporter: Colin McCabe
• Created: 16/Feb/17 19:48
• Updated: 31/Mar/17 21:19
• Resolved: 28/Feb/17 23:21

Description

The Kafka build should run findbugs to find issues that can easily be caught by static analysis.

Issue Links

• is related to: KAFKA-4826 Fix some findbugs warnings in Kafka Streams
• is related to: KAFKA-4774 Inner classes which don't need a reference to the outer class should be static
• relates to: KAFKA-4992 Investigate inconsistent synchronization of AbstractCoordinator#coordinator
• relates to: KAFKA-4796 Fix some findbugs warnings in Kafka Java client
• relates to: KAFKA-4894 Fix findbugs "default character set in use" warnings
• relates to: KAFKA-4895 Fix findbugs "format string should use %n rather than \n" in tools
• relates to: KAFKA-4897 LogCleaner#cleanSegments should not ignore failures to delete files
• relates to: KAFKA-4899 Fix findbugs warnings in kafka-core
• relates to: KAFKA-4902 Utils#delete should correctly handle I/O errors and symlinks
• relates to: KAFKA-4903 Remove unused code for reading Shell command stdout and add unit test
• relates to: KAFKA-4924 Fix findbugs warnings in Kafka-Connect-API
• relates to: KAFKA-4945 Suppress findbugs warnings about machine-generated code in jmh-benchmarks
• relates to: KAFKA-4991 KerberosLogin#login should probably be synchronized
• relates to: KAFKA-4994 Fix findbugs warning about OffsetStorageWriter#currentFlushId
• relates to: KAFKA-4995 Fix remaining findbugs warnings in Kafka Streams
• relates to: KAFKA-4996 Fix findbugs multithreaded correctness warnings for streams
• relates to: KAFKA-4944 Fix an "unread field" findbugs warning in streams examples
• relates to: KAFKA-4775 Fix findbugs warnings in kafka-tools
• links to: GitHub Pull Request #2557