Skip to content

Improve views #3744

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

Draft
wants to merge 9 commits into
base: master
Choose a base branch
from
Draft

Improve views #3744

wants to merge 9 commits into from

Conversation

rrazvan1
Copy link
Contributor

@rrazvan1 rrazvan1 commented Feb 25, 2025

Why this should be merged

  • deduplicating + cleaning and simplifying views for better readability
  • better context propagation
  • avoid corrupting intermediate nodes db

How this works

  • improving view struct
  • cleaning unused calls/extracting logic
  • batching intermediate nodes changes (when view is committed)

How this was tested

UTs

Need to be documented in RELEASES.md?

No

@rrazvan1 rrazvan1 marked this pull request as draft February 25, 2025 16:47
Copy link

This PR has become stale because it has been open for 30 days with no activity. Adding the lifecycle/frozen label will cause this PR to ignore lifecycle events.

@github-project-automation github-project-automation bot moved this to Backlog 🗄️ in avalanchego Apr 28, 2025
@rrazvan1 rrazvan1 moved this from Backlog 🗄️ to In Progress 🏗 in avalanchego Apr 28, 2025
@joshua-kim joshua-kim moved this from In Progress 🏗️ to Ready 🚦 in avalanchego May 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Ready 🚦
Development

Successfully merging this pull request may close these issues.

1 participant