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

Remove unneeded code for last_rollup tracking #5676

Open
lutter opened this issue Oct 18, 2024 · 0 comments
Open

Remove unneeded code for last_rollup tracking #5676

lutter opened this issue Oct 18, 2024 · 0 comments
Assignees
Labels
bug Something isn't working

Comments

@lutter
Copy link
Collaborator

lutter commented Oct 18, 2024

Once the changes from PR #5675 have been successfully vetted and it is clear that they won't have any undue consequences, remove the environment variable GRAPH_STORE_LAST_ROLLUP_FROM_POI and all code that is not reachable anymore.

Since we won't need the column poi2$.block_time anymore either then we might want to remove it, too. But that has to happen in a way that keeps existing subgraphs with that column in the table working.

This should be done on or around 2024-12-01

@lutter lutter added the bug Something isn't working label Oct 18, 2024
@lutter lutter self-assigned this Oct 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants
@lutter and others