[DRILL-8039] More LGTM Code Cleanup

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.19.0
• Fix Version/s: 1.20.0
• Assignee: Charles Givre
• Reporter: Charles Givre
• Created: 08/Nov/21 15:16
• Updated: 24/Nov/21 14:37
• Resolved: 24/Nov/21 14:37

Description

This PR fixes a few groups of LGTM issues:

• Hides false positive alerts for Array index out of bounds (10 alerts)
• Comparison of identical values (1 alert)
• Missing format arguments (4 alerts)
• Unused format argument (27 alerts)
• Broken or risky crypto algorithms (2 alerts)
• Implicit conversion from array to String (1 alert)
• Spurious Javadoc (2 files)

None of these affect functionality. Most relate to code maintainability and error messages.