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
Moved ProductProvenanceRetriever to FWCore/Framework #33699
Conversation
ProductProvenanceLookup is the API intended for accessing the information. ProductProvenanceRetriever now just handles the retrieving of data from storage.
This allows DataFormats/Provenance to no longer depend upon FWCore/Concurrency.
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-33699/22612
|
A new Pull Request was created by @Dr15Jones (Chris Jones) for master. It involves the following packages: DataFormats/Common @makortel, @smuzaffar, @cmsbuild, @Dr15Jones can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-06a6bd/15015/summary.html Comparison SummarySummary:
|
@wddgit please review |
@davidlange6 I hope this will help the MODULES build |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
Looks good to me. Seems like it should give identical behavior when running with these changes. I just noticed a few minor include and forward declaration issues.
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-33699/22656
|
Pull request #33699 was updated. @makortel, @smuzaffar, @cmsbuild, @Dr15Jones can you please check and sign again. |
please test |
please test the previous failures were also seen in the IBs. |
3ffc72e
to
0352af3
Compare
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-33699/22677
|
Pull request #33699 was updated. @makortel, @smuzaffar, @cmsbuild, @Dr15Jones can you please check and sign again. |
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-06a6bd/15099/summary.html Comparison SummarySummary:
|
+1 |
This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @silviodonato, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
Broke ProductProvenanceRetriever into two parts and left base interface ProductProvenanceLookup in DataFormats/Provenance. This avoids having DataFormats/Provenance depend upon FWCore/Concurrency.
PR validation:
The code compiles and the framework unit tests pass.
solves cms-sw/framework-team#141