[SOLR-10778] Address precommit WARNINGS

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 4.6
• Fix Version/s: 8.6
• Component/s: clients - java
• Assignee: Erick Erickson
• Reporter: Andrew Musselman
• Created: 31/May/17 02:01
• Updated: 15/Jul/20 15:11
• Resolved: 25/Jun/20 21:20

Description

During precommit we are seeing lots of warnings. Im turning this into an umbrella issue about getting precommit warnings out of the code in general. Yes, this will take a while.

See SOLR-10809 for getting all warnings out of solr/core. I want to selectively have precommit fail when "some part" of the code is clean so we don't backslide, and solr/core was the finest granularity I could see how to change.

If you read more of the comments here, you can see that there are some serious code refactoring that could be done. I'm electing to simply SuppressWarnings rather than re-arrange code at this point whenever the code is tricky. If anyone goes back in and tries to clean the code up, then can remove the annotation(s).

Issue Links

• incorporates: SOLR-11973 Fail compilation on warnings
• is a child of: SOLR-10810 Examine precommit lint WARNINGs in non-test code
• is a child of: SOLR-10809 Get precommit lint warnings out of Solr core
• is related to: SOLR-10779 JavaBinCodec should use close consistently rather than having marshal() and close() call finish() (which closes the underlying stream)
• is related to: LUCENE-7907 Ant precommit task WARNINGS about unclosed resources
• is related to: LUCENE-9394 Fix or suppress compile-time warnings
• relates to: LUCENE-9143 Add more static analysis and clean up resulting warnings/errors