Add an option to skip the quadratic "Package design analysis" phase

Here is a patch suggestion: when adding <sonar.skipPackageDesignAnalysis>true</sonar.skipPackageDesignAnalysis> in the <properties> element of the pom,xml, one can skip the "Package design analysis" phase.
Internally, this just skips the execution of org.sonar.plugins.squid.bridges.DesignBridge.
All comments regarding this patch are very welcome, as I don't know anything at all of Sonar's internals!

An different option would be to timebox the analysis. Meaning to stop the analysis after a certain number of cycles was found and a given maximum amount of time has been spent analyzing.

Hello,
I am facing important performance issues with the "Package Design Analysis" which is the subject of the Sonar - 2164 issue.
As far as I understand, the issue is still not resolved and the latest realeses still not support the option for switching off this analysis. When would be this feature integrated in Sonar? If this does not happen in the near future, I wonder how I can make Sonar avoid this very intensive part of the analysis ?
Thanks in advance

for information, the property "sonar.skipDesign" already allows to skip design analysis. But that's already deactivate sonar rules based on bytecode (deadcode detection, too high DIT, ...) and some metric computations (LCOM4, RFC, ...)

I am not sure what you mean by "sonar.skipDesign already allows to skip design analysis". I am running Sonar 2.5 and when setting -Dsonar.skipDesign in either local or global maven options, Sonar still keeps the Package design analysis running (which takes ages over the project I am analysing).
Do you mean that "sonar.skipDesign" has changed functionality in versions later than 2.5?

Writing this while the sonar build is in its 3th hour. Without a workaround we won't be able to use Sonar I am afraid.

Sorry Yannis, but your patch was declined, however this feature was implemented in https://github.com/SonarSource/sonar/commit/c55c3c916bd1f76f52a237e5b49eca880f8a2945

So Freddy could you please test it?

No problem !

I asked what I could do to make the patch I suggested more compliant to whatever rules you have and that I'm of course not aware of but, apparently, your changeset is not that different.
Anyway, I'm (and I guess others will) glad to see that there will be something in official releases !

Thanks again at lot for your work, guys,
Yannis

In fact Mariela Pavlova was right : bytecode might be analysed (and so package design also) even if "sonar.skipDesign" was set to "true" - see SONAR-2487.