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
Shm stream consumer 72 x #4805
Shm stream consumer 72 x #4805
Conversation
A new Pull Request was created by @smorovic (Srecko Morovic) for CMSSW_7_2_X. Shm stream consumer 72 x It involves the following packages: EventFilter/Utilities @mommsen, @cmsbuild, @Degano, @emeschi, @nclopezo can you please review it and eventually sign? Thanks. |
+1 |
This pull request is fully signed and it will be integrated in one of the next CMSSW_7_2_X IBs unless changes or unless it breaks tests. |
@@ -31,4 +34,5 @@ DEFINE_FWK_MODULE(EvFRecordInserter); | |||
DEFINE_FWK_MODULE(EvFRecordUnpacker); | |||
DEFINE_FWK_MODULE(RawStreamFileWriterForBU); | |||
DEFINE_FWK_MODULE(EvFOutputModule); |
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.
Could we remove the EvFOutputModule module declaration?
We don't want to remove it, as it intended as a "primary" name of the module. The ShmStreamConsumer (which is how the old shared-memory based output module was named) is added only due to request of TSG for supporting HLT menus generated by ConfDB. This definition is going to be removed when a new ConfDB version that doesn't have this output module name hardcoded becomes ready (reportedly in early 2015). In the meantime we want to allow configurations to be built using the new name, in fact it is already used in setups like daq2val. |
ok thanks. |
Shm stream consumer typedef (71X backport of #4805)
Typedef for the old HLT output module name, requested by TSG for ConfDB compatibility