Skip to content
Snippets Groups Projects

Have a separate routing bit writer per stream

Merged Rosen Matev requested to merge rm-stream-routing-bits into master
All threads resolved!

In the online world, all raw events go into one "pipe". They are later split into streams according to the routing bits. An event that is selected in N (>1) streams will thus get N raw events with slightly different content (e.g. different raw banks). A separate routing bit writer for each stream is thus needed in order to prevent having all of the N raw events go into each of the N streams.

  • Add/modify a test so that we have some coverage for this

This is a follow-up from the February FEST

Goes with MooreAnalysis!73 (merged)

/cc @jonrob @nskidmor

Edited by Rosen Matev

Merge request reports

Merge request pipeline #3678376 passed with warnings

Merge request pipeline passed with warnings for d3ffbc13

Approved by

Merged by Rosen MatevRosen Matev 3 years ago (Mar 9, 2022 1:27pm UTC)

Merge details

  • Changes merged into master with e9905609 (commits were squashed).
  • Deleted the source branch.

Pipeline #3680080 passed with warnings

Pipeline passed with warnings for e9905609 on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Thanks @rmatev, I just had a comment and one suggestion

  • Rosen Matev resolved all threads

    resolved all threads

  • Rosen Matev added 1 commit

    added 1 commit

    • d3ffbc13 - Apply 2 suggestion(s) to 2 file(s)

    Compare with previous version

  • Rosen Matev approved this merge request

    approved this merge request

  • merged

  • Rosen Matev mentioned in commit e9905609

    mentioned in commit e9905609

  • mentioned in commit MooreAnalysis@57b7682f

  • mentioned in issue #411 (closed)

  • mentioned in commit DaVinci@ff14b745

  • mentioned in commit DaVinci@ae892b9a

  • Please register or sign in to reply
    Loading