Fix huge memory leaks in DQM/L1TMonitorClient (port to 75x) #9894
Conversation
This whole module is a mess: lots of useless copying of th*. Should be reviewed (and possibly rewritten) by someone with proper knowledge. (cherry picked from commit 7690d8f)
|
A new Pull Request was created by @dmitrijus (Dmitrijus) for CMSSW_7_5_X. Fix huge memory leaks in DQM/L1TMonitorClient (port to 75x) It involves the following packages: DQM/L1TMonitorClient @cmsbuild, @danduggan, @deguio can you please review it and eventually sign? Thanks. |
|
+1 |
|
please test |
|
The tests are being triggered in jenkins. |
cmsbuild
added
comparison-pending
dqm-approved
fully-signed
and removed
comparison-pending
dqm-pending
orp-pending
pending-signatures
tests-pending
labels
|
This pull request is fully signed and it will be integrated in one of the next CMSSW_7_5_X IBs once checked with relvals in the development release cycle of CMSSW or unless it breaks tests. This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar |
|
This pull request is fully signed and it will be integrated in one of the next CMSSW_7_5_X IBs once checked with relvals in the development release cycle of CMSSW (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar |
|
This pull request is fully signed and it will be integrated in one of the next CMSSW_7_5_X IBs once checked with relvals in the development release cycle of CMSSW (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar |
|
+1 |
Fix huge memory leaks in DQM/L1TMonitorClient (port to 75x)
This whole module is a mess: lots of useless copying of th*.
Should be reviewed (and possibly rewritten) by someone with proper
knowledge.
(forward port, use #9893 for discussion.)