Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Apply event filter before data prefetching for OutputModules #10202

Merged
merged 5 commits into from Jul 22, 2015
Merged

Apply event filter before data prefetching for OutputModules #10202

merged 5 commits into from Jul 22, 2015

Commits on Jul 11, 2015

  1. Removed unused stream::OutputModuleBase::getTriggerResults 

    The method isn't used and the present signature does not allow for consumes.
If it becomes necessary to return this functionality, it should be done by adding a
stand alone function which is passed an EDGetTokenT<TriggerResults>.
The legacy OutputModule uses a modified version of this to support
StreamerOutputModuleBase.
    @Dr15Jones
    Dr15Jones committed Jul 11, 2015
    Copy the full SHA
    211995a View commit details
    Browse the repository at this point in the history

  2. Made OutputModule::getTriggerResults be compatible with consumes 

    The previous API for getTriggerResults would allow one to retrieve the results without having to first declare consumes. The new interface requires an EDGetTokenT<TriggerResults>. The implementation of this function also avoids coupling with the event selector code. This allowed those interfaces to be removed from TriggerResultsBasedEventSelector.
    @Dr15Jones
    Dr15Jones committed Jul 11, 2015
    Copy the full SHA
    a4b465d View commit details
    Browse the repository at this point in the history

  3. Use an event selector per Stream 

    In order to allow event selection checking to run in parallel, we needed a TriggerResultsBasedEventSelector per stream.
    @Dr15Jones
    Dr15Jones committed Jul 11, 2015
    Copy the full SHA
    9219830 View commit details
    Browse the repository at this point in the history

Commits on Jul 14, 2015

  1. Use properly initialized StreamID for test 

    Changes to the implementation of the OutputModule now require a properly initialized StreamID when calling the Event method.
    @Dr15Jones
    Dr15Jones committed Jul 14, 2015
    Copy the full SHA
    e88a8c3 View commit details
    Browse the repository at this point in the history

  2. Test for rejection of event from TriggerResults in OutputModule befor… 

    …e prefetching data

There is no reason to prefetch data and subsequently causing either delayed read or unscheduled execution if the TriggerResults being watched designate that the event should not be stored.
    @Dr15Jones
    Dr15Jones committed Jul 14, 2015
    Copy the full SHA
    8a210b2 View commit details
    Browse the repository at this point in the history