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

Port the changes in the platform API for the mirror node state cache implementation #10487

Open
bilyana-gospodinova opened this issue Feb 25, 2025 · 0 comments
Labels
enhancement Type: New feature

Comments

@bilyana-gospodinova
Copy link
Contributor

Problem

There are currently some classes that were copied from the platform API temporarily for the state cache implementation - ReadableKVStateBase and WritableKVStateBase. This works currently but is hard to maintain.

Solution

An upstream PR was created and merged - hiero-ledger/hiero-consensus-node#18012 so when it becomes a part of a release we need to remove the copied classes and use Guava's ForwardingMap and pass it in the newly added parent constructor. This way we will delegate our ScopedValue data structures from the current approach to the ForwardingMap's methods and achieve the same results but make the code more maintainable.

Alternatives

No response

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Type: New feature
Projects
Status: 📋 Backlog
Development

No branches or pull requests

1 participant