Fixes #16201: Inefficient computing of compliance on home page #2615
Conversation
|
Do not merge, need more work |
ncharles
force-pushed
the
bug_16201/inefficient_computing_of_compliance_on_home_page
branch
from
941f0df
to
61a73ce
Compare
|
Commit modified |
| @@ -455,7 +434,7 @@ trait DefaultFindRuleNodeStatusReports extends ReportingService { | |||
| } yield { | |||
| val t2 = System.currentTimeMillis | |||
| //we want to have nodeStatus for all asked node, not only the ones with reports | |||
| val nodeStatusReports = runInfos.map { case (nodeId, runInfo) => | |||
| val nodeStatusReports = runInfos.par.map { case (nodeId, runInfo) => | |||
There was a problem hiding this comment.
this part hasn't been tested yet
There was a problem hiding this comment.
but without the par, with 5000 nodes it takes 22s
There was a problem hiding this comment.
this is a linear improvement
There was a problem hiding this comment.
note for reference: this causes deadlocks - see https://issues.rudder.io/issues/16256
|
Commit modified |
ncharles
force-pushed
the
bug_16201/inefficient_computing_of_compliance_on_home_page
branch
from
61a73ce
to
6d33a8a
Compare
|
This PR is not mergeable to upper versions. |
|
OK, merging this PR |
|
OK, merging this PR |
1 similar comment
|
OK, merging this PR |
| )) | ||
| } | ||
| def getGlobalUserCompliance(): Box[Option[(ComplianceLevel, Long)]] = { | ||
| val n1 = System.currentTimeMillis |
There was a problem hiding this comment.
ha, i forgot to remove this one
|
OK, merging this PR |
https://issues.rudder.io/issues/16201