Removed need for UpdaterService in MuonDetLayerMeasurements #1402
Conversation
The UpdaterService is not thread-safe and therefore must be removed. MuonDetLayerMeasurments was the only user of UpdaterService in production jobs. The use of the UpdaterService was replaced with the new cacheIdentifier() information from the edm::Event. The cacheIdentifier() can be used to quickly check to see if an edm::Event has changed since the last call to a method.
|
@ktf Giulio, why hasn't this been assigned to a L2? |
|
A new Pull Request was created by @Dr15Jones (Chris Jones) for CMSSW_7_0_X. Removed need for UpdaterService in MuonDetLayerMeasurements It involves the following packages: RecoMuon/MeasurementDet @nclopezo, @cmsbuild, @thspeer, @slava77 can you please review it and eventually sign? Thanks. |
|
glitch. |
|
+1 |
|
This pull request is fully signed and it will be integrated in one of the next IBs unless changes or unless it breaks tests. @ktf can you please take care of it? |
…etLayerMeasurements Multithreading fixes -- Removed need for UpdaterService in MuonDetLayerMeasurements
Dr15Jones
deleted the
removeUpdaterServiceFromMuonDetLayerMeasurements
branch
The UpdaterService is not thread-safe and therefore must be removed. MuonDetLayerMeasurments was the only user of UpdaterService in production jobs. The use of the UpdaterService was replaced with the new cacheIdentifier() information from the edm::Event. The cacheIdentifier() can be used to quickly check to see if an edm::Event has changed since the last call to a method.