Fix Spotbugs check failed

[hangc0276]

Motivation

The Spotbugs check failed, which blocked the PRs' CI

• https://github.com/apache/bookkeeper/actions/runs/4315781701/jobs/7530640684
• https://github.com/apache/bookkeeper/actions/runs/4319029437/jobs/7538019260
• When generating javadoc, it throws NPE:

[ERROR] NullPointerException
java.lang.NullPointerException
    at org.apache.maven.lifecycle.internal.MojoExecutor.executeForkedExecutions (MojoExecutor.java:439)
    at org.apache.maven.lifecycle.internal.MojoExecutor.doExecute (MojoExecutor.java:325)
    at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:213)
    at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:175)
    at org.apache.maven.lifecycle.internal.MojoExecutor.access$000 (MojoExecutor.java:76)
    at org.apache.maven.lifecycle.internal.MojoExecutor$1.run (MojoExecutor.java:163)
    at org.apache.maven.plugin.DefaultMojosExecutionStrategy.execute (DefaultMojosExecutionStrategy.java:39)
    at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:160)
    at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject (LifecycleModuleBuilder.java:105)
    at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject (LifecycleModuleBuilder.java:73)
    at org.apache.maven.lifecycle.internal.builder.singlethreaded.SingleThreadedBuilder.build (SingleThreadedBuilder.java:53)
    at org.apache.maven.lifecycle.internal.LifecycleStarter.execute (LifecycleStarter.java:118)
    at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:260)
    at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:172)
    at org.apache.maven.DefaultMaven.execute (DefaultMaven.java:100)
    at org.apache.maven.cli.MavenCli.execute (MavenCli.java:821)
    at org.apache.maven.cli.MavenCli.doMain (MavenCli.java:270)
    at org.apache.maven.cli.MavenCli.main (MavenCli.java:192)
    at jdk.internal.reflect.NativeMethodAccessorImpl.invoke0 (Native Method)
    at jdk.internal.reflect.NativeMethodAccessorImpl.invoke (NativeMethodAccessorImpl.java:62)
    at jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke (DelegatingMethodAccessorImpl.java:43)
    at java.lang.reflect.Method.invoke (Method.java:566)
    at org.codehaus.plexus.classworlds.launcher.Launcher.launchEnhanced (Launcher.java:282)
    at org.codehaus.plexus.classworlds.launcher.Launcher.launch (Launcher.java:225)
    at org.codehaus.plexus.classworlds.launcher.Launcher.mainWithExitCode (Launcher.java:406)
    at org.codehaus.plexus.classworlds.launcher.Launcher.main (Launcher.java:347)

It is caused by a maven 3.9.0: https://issues.apache.org/jira/browse/MNG-7679

However, our GitHub CI uses maven 3.9.0 by default

Modifications

• Fix Spotbugs check failed.
• Set GitHub CI's maven version to 3.8.7

[codecov-commenter]

Codecov Report

Merging #3836 (efc0674) into master (227b166) will increase coverage by 20.80%.
The diff coverage is 94.11%.

@@              Coverage Diff              @@
##             master    #3836       +/-   ##
=============================================
+ Coverage     47.52%   68.32%   +20.80%     
- Complexity     4579     6763     +2184     
=============================================
  Files           473      473               
  Lines         40954    40950        -4     
  Branches       5239     5240        +1     
=============================================
+ Hits          19462    27981     +8519     
+ Misses        19498    10712     -8786     
- Partials       1994     2257      +263     

Flag	Coverage Δ
bookie	39.84% <35.29%> (?)
client	44.15% <94.11%> (?)
remaining	29.59% <52.94%> (+0.08%)	⬆️
replication	41.31% <82.35%> (-0.16%)	⬇️
tls	21.01% <35.29%> (+0.09%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
...apache/bookkeeper/client/DistributionSchedule.java	9.09% <ø> (+9.09%)	⬆️
...java/org/apache/bookkeeper/util/PageCacheUtil.java	53.12% <0.00%> (ø)
...va/org/apache/bookkeeper/client/LedgerChecker.java	81.55% <100.00%> (+3.77%)	⬆️
...ava/org/apache/bookkeeper/client/PendingAddOp.java	87.12% <100.00%> (+24.02%)	⬆️
...rg/apache/bookkeeper/client/PendingWriteLacOp.java	73.17% <100.00%> (+73.17%)	⬆️
...kkeeper/client/RoundRobinDistributionSchedule.java	84.73% <100.00%> (+32.86%)	⬆️
...keeper/replication/AuditorCheckAllLedgersTask.java	53.74% <0.00%> (-4.09%)	⬇️
...va/org/apache/bookkeeper/versioning/Versioned.java	75.00% <0.00%> (ø)
...org/apache/bookkeeper/client/BookieInfoReader.java	70.81% <0.00%> (ø)
...apache/bookkeeper/meta/ZkLedgerAuditorManager.java	71.69% <0.00%> (ø)
... and 236 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more