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

Delete consensus data from tables when it is no longer needed (#19697) · MystenLabs/sui@fedfb0d #4678

Closed
bingyanglin opened this issue Jan 6, 2025 · 1 comment · Fixed by #5053
Assignees
Labels
good first issue Good for newcomers node Issues related to the Core Node team

Comments

@bingyanglin
Copy link
Contributor

Delete consensus data from tables when it is no longer needed (#19697) · MystenLabs/sui@fedfb0d

refactory to remove consensus data from tables when it is no longer needed.

@bingyanglin bingyanglin added node Issues related to the Core Node team good first issue Good for newcomers labels Jan 6, 2025
@gokhan-simsek-iota gokhan-simsek-iota self-assigned this Jan 20, 2025
@gokhan-simsek-iota
Copy link
Contributor

gokhan-simsek-iota commented Jan 21, 2025

In short: Not applicable without investigating/changing to newer code.

Details:
These upstream changes require these and related newer version objects and more dependent code changes:

assigned_shared_object_versions_v2
pending_checkpoints_v2

So this one does not seem to be a simple task, and does not seem that we cannot apply these changes without investigating/changing to newer code.

Next Actions:

  • Investigate and apply required upstream changes and then this change.
  • Or do not merge this and close this issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers node Issues related to the Core Node team
Projects
None yet
2 participants