-
Notifications
You must be signed in to change notification settings - Fork 20
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
DBouputService migration #28
Comments
To check which packages are still using the deprecated call with pointers instead of reference. Here are those call to writeOne:
All .cc files calling createNewIOV:
All .cc files calling "writeForNextLumisection":
(none for writeMany) Will check out these packages, remove DB old method and see which fails to build. |
By checking out the CMSSW_12_1_0_pre3, we have the latest DB code. I comment out the old writeOne API such that those not updated from the deprecated method will fail to build. I grab the master CMSSW source tarball and copy the DB related packages over and compile. So far the following ones which are still calling the old writeOne method need to be updated:
(done) For .cc files calling old createNewIOV method:
(done) For .cc files calling "writeForNextLumisection": |
There is a partial fix in cms-sw/cmssw#35616 |
First of the subsystem PR done in cms-sw/cmssw#35999 (for SiStrips) |
Hi @mmusich I found two more files for Strips |
While https://github.com/cms-sw/cmssw/blob/master/CalibTracker/SiStripChannelGain/src/SiStripGainsPCLHarvester.cc#L136 is already OK. It uses |
I made a PR for the SiPixelCPE cms-sw/cmssw#36027 I let the rest to @SanjanaSekhar |
DropBox metadata (among other things) cms-sw/cmssw#36028 |
Remaining SiPixel done in cms-sw/cmssw#36040 |
ECAL is done in cms-sw/cmssw#36062 |
Beamspot done in cms-sw/cmssw#36077 |
TkAlignment done in cms-sw/cmssw#36076 |
AlCa part done in cms-sw/cmssw#36090 |
The removal of the old method happened in cms-sw/cmssw#36490 , with this, this migration is completed! |
Original issue: cms-sw/cmssw#28699
In cms-sw/cmssw#35048 some methods in
CondCore/DBOutputService
were changed to accept pointers instead of references.All calls to the following methods should migrated before the old methods (that still accept references and were left in the code in order not to break the release) can be removed:
The PR cms-sw/cmssw#35556 then renames the new method
gDoc for the files: https://docs.google.com/spreadsheets/d/1DJ6u45mBXW0bfYcdNNzauIT9nlWlyyjWZhKPuOmpB7U/edit#gid=1265504621
To do list:
The text was updated successfully, but these errors were encountered: