Skip to content
Snippets Groups Projects

[TrigOutputHandling] Skip serialisation of Trigger EDM in events where HLT output is not written out

Merged Rafal Bielski requested to merge rbielski/athena:remove-force-serialise into 22.0

Remove an unneeded "safety net" feature which was based on a wrong assumption that all HLT accepted events need to write at least one HLT ROBFragment to the output stream. That is not the case for events accepted only to detector calibration PEB streams which don't include HLT result, for example calibration_LArPEBDigitalTrigger or calibration_SCTNoise.

This will save some CPU cycles by skipping the unnecessary serialisation in these cases. Also removes unnecessary warnings from logs at P1.

Closes ATR-25227 as it was the last leftover to-do item in that ticket. Coincidentally this unneeded feature saved us from more serious fallout of the bug reported in the ticket, but since the bug is fixed the feature can be removed.

FYI @wiedenma, @astruebi, @palacino

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
Please register or sign in to reply
Loading