You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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
The text was updated successfully, but these errors were encountered:
Problem
There are currently some classes that were copied from the platform API temporarily for the state cache implementation -
ReadableKVStateBase
andWritableKVStateBase
. 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 ourScopedValue
data structures from the current approach to theForwardingMap's
methods and achieve the same results but make the code more maintainable.Alternatives
No response
The text was updated successfully, but these errors were encountered: