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

Evaluate user needs for presence of storage fields #128

Open
thpani opened this issue Sep 24, 2024 · 0 comments
Open

Evaluate user needs for presence of storage fields #128

thpani opened this issue Sep 24, 2024 · 0 comments
Labels
enhancement New feature or request

Comments

@thpani
Copy link
Collaborator

thpani commented Sep 24, 2024

The presence of storage fields is different between instance and persistent/temporary storage:
All instance data is stored in a single LedgerEntry, thus all instance fields are present if a single instance field changes.
persistent/temporary have one LedgerEntry per field, thus they are only present in the transaction meta if that field changes.

Currently (#123), this behavior is mirrored by Solarkraft, i.e., persistent/temporary are only present as storage fields if they are modified by the transaction.

We'll need one (or a few) case studies to determine if the current behavior is enough, or if we indeed need access to fields untouched by the transaction. If the latter is the case, we'll need to

  1. make the monitors stateful (accumulating historical transactions), or
  2. proactively fetch all persistent/temporary fields (an open question is how we'd know which fields exist)
@thpani thpani added this to the M6: DevRel and case studies milestone Sep 24, 2024
@thpani thpani changed the title Evaluate user needs for presence of storage fields. Evaluate user needs for presence of storage fields Sep 24, 2024
@thpani thpani added the enhancement New feature or request label Oct 23, 2024
@thpani thpani removed this from the M6: DevRel and case studies milestone Oct 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant