Skip to content
Snippets Groups Projects

MuonCondAlg: Fix memory leak in NswCalibDbAlg

Merged Alaettin Serhan Mete requested to merge amete/athena:22.0-nswleak into 22.0

This MR fixes the massive memory leak in NswCalibDbAlg.

Closes ATLASRECTS-7083 and relates to ATR-25529

Edit: Now the algorithm doesn't do anything if the write handles are still valid so that we don't end up executing unnecessary code in serial jobs.

Edited by Alaettin Serhan Mete

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
  • Johannes Junggeburth resolved all threads

    resolved all threads

  • added 1 commit

    • 00bc7281 - More improvements to NswCalibDbAlg

    Compare with previous version

  • Alaettin Serhan Mete resolved all threads

    resolved all threads

  • Alaettin Serhan Mete changed the description

    changed the description

  • This merge request affects 1 package:

    • MuonSpectrometer/MuonConditions/MuonCondGeneral/MuonCondAlg

    This merge request affects 1 file:

    • MuonSpectrometer/MuonConditions/MuonCondGeneral/MuonCondAlg/src/NswCalibDbAlg.cxx

    Adding @stavrop ,@egodden ,@rosati ,@wleight ,@jojungge as watchers

  • :white_check_mark: CI Result SUCCESS (hash 00bc7281)

    Athena DetCommon
    externals :white_check_mark: :white_check_mark:
    cmake :white_check_mark: :white_check_mark:
    make :white_check_mark: :white_check_mark:
    required tests :white_check_mark: :white_check_mark:
    optional tests :white_check_mark: :white_check_mark:

    Full details available on this CI monitor view. Check the JIRA CI status board for known problems
    :white_check_mark: Athena: number of compilation errors 0, warnings 0
    :white_check_mark: DetCommon: number of compilation errors 0, warnings 0
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 53751]

  • Re-approving after latest changes (which look good as well).

    Jakub (L1 shifters)

  • added review-approved label and removed review-pending-level-1 label

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading