-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Fix concurrent lumi usage in DQMStore [12_0] #35752
Fix concurrent lumi usage in DQMStore [12_0] #35752
Conversation
Avoid deleting the MonitorElement meta data for localMEs
A new Pull Request was created by @Dr15Jones (Chris Jones) for CMSSW_12_0_X. It involves the following packages:
@emanueleusai, @ahmad3213, @cmsbuild, @jfernan2, @pmandrik, @pbo0, @rvenditti can you please review it and eventually sign? Thanks. cms-bot commands are listed here
|
Backport of #35751 |
please test |
type bug-fix |
urgent |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-98ca7b/19779/summary.html Comparison SummarySummary:
|
hold |
Pull request has been put on hold by @Dr15Jones |
@Dr15Jones now that #35757 is merged in 12_0, can we close this (to clean up the queue)? |
superseded by #35757 |
PR description:
Avoid deleting the MonitorElement meta data for localMEs.
Backport of #35751.
PR validation:
Code compiles and simple job which was failing before now succeeds. Also varified with valgrind.