SonarQube
  1. SonarQube
  2. SONAR-2164

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

    Details

    • Type: Improvement Improvement
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.5
    • Fix Version/s: 2.9
    • Component/s: DSM
    • Labels:
      None
    • Number of attachments :
      1

      Description

      For large projects and/or projects with bad separation of concerns between packages, the "Package design analysis" phase can take ages and totally prevent using Sonar.
      An option should be added to skip this phase or at least skip the quadratic analysis behind it.

      JIRA opened from discussion at http://old.nabble.com/%22Package-design-analysis%22-is-too-slow-%28been-running-for-4-days-already...%29-td30833804.html.

        Issue Links

          Activity

          Hide
          Sasha Ovsankin added a comment -

          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.

          Show
          Sasha Ovsankin added a comment - 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.
          Hide
          Evgeny Mandrikov added a comment -

          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?

          Show
          Evgeny Mandrikov added a comment - 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?
          Hide
          Yannis BRES added a comment -

          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

          Show
          Yannis BRES added a comment - 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
          Hide
          Freddy Mallet added a comment -

          Tested

          Show
          Freddy Mallet added a comment - Tested
          Hide
          Evgeny Mandrikov added a comment -

          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.

          Show
          Evgeny Mandrikov added a comment - 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 .

            People

            • Assignee:
              Evgeny Mandrikov
              Reporter:
              Yannis BRES
            • Votes:
              5 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: