-
Notifications
You must be signed in to change notification settings - Fork 320
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
chore: byte index to read singularEvent, propagate payload func to processor pipeline #5314
base: master
Are you sure you want to change the base?
Conversation
77224ff
to
d5f7134
Compare
…c to processor pipeline
2854a3f
to
e666034
Compare
e666034
to
b0d7048
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #5314 +/- ##
==========================================
- Coverage 73.57% 73.54% -0.03%
==========================================
Files 424 424
Lines 59935 59961 +26
==========================================
+ Hits 44095 44098 +3
- Misses 13370 13397 +27
+ Partials 2470 2466 -4 ☔ View full report in Codecov by Sentry. |
This PR is considered to be stale. It has been open 20 days with no further activity thus it is going to be closed in 7 days. To avoid such a case please consider removing the stale label manually or add a comment to the PR. |
This PR is considered to be stale. It has been open 20 days with no further activity thus it is going to be closed in 7 days. To avoid such a case please consider removing the stale label manually or add a comment to the PR. |
Description
Use byte index to get event payload and propagate same to processor pipeline steps - using
sync.OnceValue
instead ofgo-kit/memoize
so it can be propagated to other steps. This can be used in setting report payloads or settings event payloads to the error db.Also updated tests to remove cases where processor picked up jobs from gateway that had multiple events - this hasn't been the case for a long time now.
Linear Ticket
Resolves PIPE-1717
Security