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
[12_0_X] Better handling of timeout in FWCore/SharedMemory #35076
[12_0_X] Better handling of timeout in FWCore/SharedMemory #35076
Conversation
The call to timed_wait will still take just as long as it takes the Worker to complete but will then return false if that time was longer than the set wait time. Now we use transitionID_ to signify if the worker did the work and if it did, we ignore the timeout. Added new `retry` methods which allow user control of what to do if a timeout does happen.
A new Pull Request was created by @colizz (Congqiao Li) for CMSSW_12_0_X. It involves the following packages:
@makortel, @smuzaffar, @cmsbuild, @Dr15Jones can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
@cmsbuild, please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-99b21d/18143/summary.html Comparison SummarySummary:
|
+1 |
This pull request is fully signed and it will be integrated in one of the next CMSSW_12_0_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_12_1_X is complete. This pull request will now be reviewed by the release team before it's merged. @perrotta, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
Backport of #34815. (commits are cherry-picked)
The backport is needed in 12_0_X because some POG samples will use concurrent GEN utilities, with Run 3 condition under this environment.
PR validation:
Unit tests work.