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

refactor: Remove the availability oracle #8471

Closed
Tracked by #7820
LHerskind opened this issue Sep 10, 2024 · 0 comments · Fixed by #8474
Closed
Tracked by #7820

refactor: Remove the availability oracle #8471

LHerskind opened this issue Sep 10, 2024 · 0 comments · Fixed by #8474
Assignees

Comments

@LHerskind
Copy link
Contributor

LHerskind commented Sep 10, 2024

With changes planned for progressing to using blobs, we do not really need the availability oracle as it is.

I suggest that we get rid ahead of the chance since that would also aling the code path towards something that will look more like the blobs.

Namely, currently it is possible to publish the data in one call, and then a separate propose a block. Related to #8452 it would make sense to entirely remove that separate flow as it is not intended to be used with blobs anyway.

Our intended use is that the propose will also publish the data, so moving to that now should make the transition simpler.

At the same time, it should provide us with a simpler archiver that skips a lot of the "double" work it is currently doing, because it is fetching metadata and block content in separate fetchers, but they would always be provided at the same time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant