NavGraph: Remove ElementLink, replace with Decision pointer
Long wait time (~100ms per event (CPU time, 8 threads)) in decisionToElementLink function was observed, due to mutex lock in T2pMap. However, NavGraph node does not need ElementLink of Decision. It can be replaced with pointer to Decision, which avoids mutex lock.
Many files are edited because EventContext is no longer needed as a part of argument.
Merge request reports
Activity
This merge request affects 5 packages:
- Trigger/TrigAnalysis/TrigDecisionTool
- Trigger/TrigAnalysis/TrigNavSlimmingMT
- Trigger/TrigSteer/DecisionHandling
- Trigger/TrigSteer/TrigCompositeUtils
- Trigger/TrigSteer/ViewAlgs
Affected files list will not be printed in this case
Adding @tamartin as watcher
added 24.0 Trigger analysis-review-required review-pending-level-1 labels
✅ CI Result SUCCESS (hash 1dd297b4)Athena AnalysisBase AthAnalysis externals ✅ ✅ ✅ cmake ✅ ✅ ✅ make ✅ ✅ ✅ tests ✅ ✅ ✅ Full details available on this CI monitor view. Check the JIRA CI status board for known problems
✅ Athena: number of compilation errors 0, warnings 0
✅ AnalysisBase: number of compilation errors 0, warnings 0
✅ AthAnalysis: number of compilation errors 0, warnings 0
📝 For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 14507] (remote access info)added review-approved label and removed review-pending-level-1 label
Relevant to ATR-26805.
added review-approved-point1 label
@ruyuan n for analysis review
- Resolved by Stefanie Morgenstern
Should be no analysis impact to this one - a pure refactor.
mentioned in commit d4e298eb
mentioned in merge request !73803 (closed)
mentioned in merge request !73803 (closed)
added analysis-review-approved label and removed analysis-review-required label
mentioned in merge request !73815 (merged)
mentioned in merge request !73872 (merged)