-
Notifications
You must be signed in to change notification settings - Fork 363
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
v0.9.8 total duration drastically increased #1310
Comments
Hi @tomikais,
Regards, |
Hi @guwirth, Did you make a one to one comparison: Can you ensure that there are no bigger change sin your code base and the report generation?
Another question: What was the time before?
Turn debug info on to get more information what is going on internally.
Maybe you can turn CXX checks off to see if one of the checks (e.g. Cognitive complexity) or the parser itself is the root cause.
Thanks, Thomas |
@tomikais 11509369ms are 3,2h: how can it be 2 or 3h less?
Try in Administration / Projects / Background Tasks |
@ALL someone else have problems? |
Result:
Fix: |
@tomikais please have a look here Retrieving compiler information from buildlog https://github.com/SonarOpenCommunity/sonar-cxx/wiki/Compilers. Parsing for warnings and deriving defines and includes is combined. Maybe this should be separated? |
@tomikais could you please turn debug info on and have a look how often your LOG file is parsed to get the defines and includes. Thx. |
@tomikais The compiler reports are handled in a different step and are not releated to SquidSensor. A huge impact on the source code processing for the AST are the settings for system include folders which are extracted from the compiler report files. Please check the the settings for scanner context on the SonarQube server (Administration->Projects->Background Tasks).
|
@Bertk my first thought was that that the LOG file is read only once to parse the warnings and the configuration in one step. But as you are stating these steps are independent. As a consequence I would split the configuration setting to make this more transparent. The Json Compilation Database is more or less the same and functionality should be somehow combined. The https://github.com/SonarOpenCommunity/sonar-cxx/blob/master/cxx-squid/src/main/java/org/sonar/cxx/CxxVCppBuildLogParser.java |
@guwirth CxxVCppBuildLogParser is called whenever a source file is analyzed and this is similar to |
2017.11.16 10:48:40 INFO [o.s.s.c.t.CeWorkerCallableImpl] Execute task | project=heidelberg.view2connect:trunk | type=REPORT | id=AV_EOvrl_pFO7sgivDyG |
close this, there is the issue #1325 to track it |
hi @guwirth ,
after updating the plugin to version 0.9.8 on our SonarQube v5.6.6 server the total duration drastically increased.
log: INFO: Sensor c++ SquidSensor (done) | time=11509369ms
Can you please help me to identify the problem.
Kind regards, Thomas
The text was updated successfully, but these errors were encountered: