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.

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

Event Publishing Be An Array Endpoint #114

Open
hannahhoward opened this issue Aug 5, 2022 · 1 comment
Open

Event Publishing Be An Array Endpoint #114

hannahhoward opened this issue Aug 5, 2022 · 1 comment
Assignees

Comments

@hannahhoward
Copy link
Collaborator

Goal

Building on #112 and in conjunction with filecoin-project/tornado-deploy#8, let's make the retrieval events posting api accept an array of events, instead of a single event. Based on the architecture of #112, it should be possible to just make the reportChan on the event recorder an array of reports, and then in the ingestReports go routine just forward the whole array of queued events rather than a single event.

We'll need to figure out timing so we can launch this at the same time as filecoin-project/tornado-deploy#8

@kylehuntsman
Copy link
Collaborator

The API could accept both formats as a compatibility thing, and then we would just need the API to be deployed prior to the client update.

@hannahhoward hannahhoward self-assigned this Aug 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants