Branch: refs/heads/spotbugs-fixes
  Home:   https://github.com/jenkinsci/gearman-plugin
  Commit: a408cb5e9a63037313ef8cff09c2e102dc7cd074
      
https://github.com/jenkinsci/gearman-plugin/commit/a408cb5e9a63037313ef8cff09c2e102dc7cd074
  Author: Antoine Musso <has...@free.fr>
  Date:   2022-01-17 (Mon, 17 Jan 2022)

  Changed paths:
    M src/main/java/hudson/plugins/gearman/GearmanProxy.java

  Log Message:
  -----------
  GearmanProxy: avoid NPE if a finished build has no executor

When the build is finalized we retriever the executor/computer that ran
it to get other works to check the availability lock. Spotbugs
complains `Run.getExecutor()` might be null which really should never
happen.  If that is ever the case, throw an IllegalStateException, it
will be caught and reported by Jenkins runListener.


  Commit: e031ce9435b391f35fdc2e8a2c55bd15d46d7d1f
      
https://github.com/jenkinsci/gearman-plugin/commit/e031ce9435b391f35fdc2e8a2c55bd15d46d7d1f
  Author: Antoine Musso <has...@free.fr>
  Date:   2022-01-17 (Mon, 17 Jan 2022)

  Changed paths:
    M src/main/java/hudson/plugins/gearman/MyGearmanWorkerImpl.java

  Log Message:
  -----------
  MyGearmanWorkerImpl: fix RV_RETURN_VALUE_IGNORED_BAD_PRACTICE

Spotbugs complains about ExecutorService.submit() return value being
ignored. The returned Future is unneeded, just assign it to a variable
and forget about it.


  Commit: 2d17533016346d374dbf8748f3a9d609e7ac6f2e
      
https://github.com/jenkinsci/gearman-plugin/commit/2d17533016346d374dbf8748f3a9d609e7ac6f2e
  Author: Antoine Musso <has...@free.fr>
  Date:   2022-01-17 (Mon, 17 Jan 2022)

  Changed paths:
    M pom.xml

  Log Message:
  -----------
  build: fail again on findbug warnings

Findbugs analyzis was no more enforced since 7e7e3f12d, they should all
now be fixed.


Compare: 
https://github.com/jenkinsci/gearman-plugin/compare/a441abe776fc...2d1753301634

-- 
You received this message because you are subscribed to the Google Groups 
"Jenkins Commits" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to jenkinsci-commits+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/jenkinsci-commits/jenkinsci/gearman-plugin/push/refs/heads/spotbugs-fixes/a441ab-2d1753%40github.com.

Reply via email to