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
DBOutputService: Removed deprecated methods #36466
DBOutputService: Removed deprecated methods #36466
Conversation
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-36466/27320
|
A new Pull Request was created by @ggovi for master. It involves the following packages:
@malbouis, @yuanchao, @cmsbuild, @ggovi, @francescobrivio, @tvami can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test |
-1 Failed Tests: Build HeaderConsistency BuildI found compilation error when building: 106 | static const std::string& extendedBaseType(EDAnalyzer const*) { return kExtendedBaseForEDAnalyzer; } | ^ >> Compiling edm plugin /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_12_3_X_2021-12-12-0000/src/DQM/DTMonitorClient/src/DTSummaryClients.cc In file included from /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_12_3_X_2021-12-12-0000/src/DQM/DTMonitorClient/src/DTLocalTriggerSynchTest.cc:24: /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_12_3_X_2021-12-12-0000/src/CalibMuon/DTCalibration/interface/DTCalibDBUtils.h: In static member function 'static void DTCalibDBUtils::writeToDB(std::string, T*)': /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_12_3_X_2021-12-12-0000/src/CalibMuon/DTCalibration/interface/DTCalibDBUtils.h:35:24: error: 'class cond::service::PoolDBOutputService' has no member named 'writeOne'; did you mean 'writeOneIOV'? 35 | dbOutputSvc->writeOne(payload, dbOutputSvc->beginOfTime(), record); | ^~~~~~~~ | writeOneIOV /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_12_3_X_2021-12-12-0000/src/CalibMuon/DTCalibration/interface/DTCalibDBUtils.h:35:34: error: expected primary-expression before '>' token 35 | dbOutputSvc->writeOne(payload, dbOutputSvc->beginOfTime(), record); |
after merging of #36366 this needs a rebase (file |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-36466/27334
|
Pull request #36466 was updated. @malbouis, @yuanchao, @pmandrik, @emanueleusai, @ahmad3213, @tvami, @cmsbuild, @ggovi, @jfernan2, @clacaputo, @slava77, @jpata, @francescobrivio, @pbo0, @rvenditti can you please check and sign again. |
@cmsbuild please test |
Please rebase on top of latest CMSSW_12_3_X_2021-12-13-2300, which contains the conflicting PR #36366 |
-1 Failed Tests: UnitTests Unit TestsI found errors in the following unit tests: ---> test testCondToolsSiStripBuildersReaders had ERRORS Comparison SummarySummary:
|
Replaced by #36490 |
-1
|
PR description:
The PoolDBoutputService interface (and its Derived class OnlineDBOutputService ) contains methods that have been deprecated in favour of new methods providing the same functionalities with a more explicit signature concerning the payload object ownership policy. Since all of the clients of the old interface in CMSSW have been migrated to the new interface, in this PR we proceed to remove the deprecated methods.
PR validation:
Unit and integration tests