search

amitdev / PMD-Intellij

Plugin for doing static analysis in Intellij using PMD
http://plugins.intellij.net/plugin/?id=1137
MIT License
71 stars 23 forks source link

Number of threads used should be default to number of available processors and should be validated #166

Closed jborgers closed 7 months ago

jborgers commented 7 months ago

The number of threads used by pmd has a big impact on the speed of the pmd run. In my test configuring 1 thread resulted in about 21 s. taken and with 8 threads (I have 8 hardware threads) the pmd run took only 9 s.

The default of 1 thread is a slow choice, default should be Runtime.availableProcessors, just like PMD itself has its default. Adding more threads has no use, as in my tests it shows the process is CPU bound, and the pmd run took longer with threads > avaliableProcs.

jborgers commented 7 months ago

Also make clear in the UI what fast setting is (8 in my case)

jborgers commented 7 months ago

works in 1.9.2