Make Worker only if module uses the transition - #51634
Conversation
If a module doesn't process a stream Run or LuminosityBlock transition then we no longer make a Worker for that transition.
|
please test |
|
cms-bot internal usage |
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-51634/50469
|
|
A new Pull Request was created by @Dr15Jones for master. It involves the following packages:
@Dr15Jones, @makortel, @smuzaffar can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
|
+1 Size: This PR adds an extra 52KB to repository The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
You can see more details here: Comparison SummarySummary:
Max Memory Comparisons exceeding threshold@cms-sw/core-l2 , I found 6 workflow step(s) with memory usage exceeding the error threshold: Expand to see workflows ...
|
|
Changes look fine, but I'd like to run the tests again in the next IB to have clean comparisons |
|
@cmsbuild, please test |
|
+1 Size: This PR adds an extra 16KB to repository Comparison SummarySummary:
|
|
+core |
|
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. @ftenchini, @sextonkennedy, @mandrenguyen (and backports should be raised in the release meeting by the corresponding L2) |
|
+1 |
PR description:
If a module doesn't process a stream Run or LuminosityBlock transition then we no longer make a Worker for that transition.
PR validation:
Code compiles and all framework unit tests pass.
resolves cms-sw/framework-team#2378