-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
[KS-194] Send Streams reports to TriggerService #13203
Conversation
I see you updated files related to
|
// NOTE: Skipping fields derived from FullReport, they will be filled out at a later stage | ||
// after decoding and validating signatures. | ||
} | ||
return mt.triggerCapability.ProcessReport([]capMercury.FeedReport{converted}) |
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.
We are assuming this is async/non-blocking?
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.
Yes, non-blocking in a networking sense. It only saves the report in a local map under lock - implementation here: https://github.com/smartcontractkit/chainlink-common/blob/main/pkg/capabilities/triggers/mercury_trigger.go#L82
Quality Gate passedIssues Measures |
No description provided.