-
Notifications
You must be signed in to change notification settings - Fork 9.2k
HADOOP-19731. Fix SpotBugs warnings introduced after SpotBugs version upgrade. #8053
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: trunk
Are you sure you want to change the base?
Conversation
36547f5 to
c46c6af
Compare
|
@zhtttylz Thank you for following up on this issue. |
|
💔 -1 overall
This message was automatically generated. |
|
💔 -1 overall
This message was automatically generated. |
366bf41 to
7b27c79
Compare
|
@steveloughran @szetszwo We propose adding a global configuration to temporarily suppress the warnings generated by the new rules introduced in SpotBugs 4.9.7. Would this approach be acceptable to you? |
@slfan1989, @zhtttylz , it sounds good. Thanks a lot for fixing the warnings! |
Description of PR
HADOOP-19731. Fix SpotBugs warnings introduced after SpotBugs version upgrade.
How was this patch tested?
Ran
mvn -Dspotbugs.skip=false spotbugs:spotbugson affected modules and verified the build no longer fails on SpotBugs warnings. No functional code changes, config-only.For code changes: