Fix reviews

Author: Vlatombe

core/src/main/java/hudson/model/AsyncPeriodicWork.java

@@ -92,7 +92,7 @@ public final void doRun() {
         try {
             if (thread != null && thread.isAlive()) {
                 if (queueIfAlreadyRunning()) {
-                    logger.log(Level.FINE, "Scheduling another run of {0} since it is already running", name);
+                    logger.log(this.getSlowLoggingLevel(), "Scheduling another run of {0} since it is already running", name);
                     pending.set(true);
                 } else {
                     logger.log(this.getSlowLoggingLevel(), "{0} thread is still running. Execution aborted.", name);
@@ -121,7 +121,7 @@ public final void doRun() {
                 logger.log(Level.FINE, "Finished {0}. {1,number} ms",
                         new Object[]{name, stopTime - startTime});
                 thread = null;
-                if (queueIfAlreadyRunning() && pending.getAndSet(false)) {
+                if (pending.getAndSet(false)) {
                     logger.log(this.getSlowLoggingLevel(), "An execution of {0} was requested while it was running, scheduling another run now", name);
                     doRun();
                 }

test/src/test/java/hudson/model/AsyncPeriodicWorkTest.java

@@ -39,7 +39,7 @@ public void extraCallGetsQueued() {
         await("The second call has been queued and executed later").until(instance::getCount, is(2));
     }
 
-    @TestExtension({"extraCallGetsIgnored", "extraCallGetsQueued"})
+    @TestExtension
     public static class AsyncPeriodicWorkTestImpl extends AsyncPeriodicWork {
         private boolean queueIfAlreadyRunning;
         private int count = 0;