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.

Sign up for GitHub

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

Jump to bottom

Remove DataColumnInfo #6572

Open
eserilev opened this issue Nov 7, 2024 · 1 comment
Open

Remove DataColumnInfo #6572

eserilev opened this issue Nov 7, 2024 · 1 comment
Labels
das Data Availability Sampling database optimization Something to make Lighthouse run more efficiently.

Comments

@eserilev
Copy link
Collaborator

eserilev commented Nov 7, 2024

Description

In #6559 were going ahead and removing BlobInfo. Its a bit of a pain to manage as it requires us to make an atomic transaction across two separate databases. Instead we're going to just iterate across the full blob db and prune blobs past the data availability window (additional info can be found in the linked issue)

I'd like to do the same for DataColumnInfo. Unless theres some additional need for this table that I'm unaware of, I'd like to go ahead and remove it.
@eserilev eserilev added optimization Something to make Lighthouse run more efficiently. database das Data Availability Sampling labels Nov 7, 2024
@eserilev eserilev changed the title remove DataColumnInfo Remove DataColumnInfo Nov 7, 2024
@michaelsproul
Copy link
Member

Let's do it 🔥

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
das Data Availability Sampling database optimization Something to make Lighthouse run more efficiently.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@michaelsproul @eserilev