Draft
Conversation
Test Results for Commit 77ba2d7Pull Request 4981: Results Test Case Results
Last updated: 2026-02-22 05:20:34 UTC |
2dcdb2c to
6d4a2ed
Compare
40e2e6f to
28b8749
Compare
22a66c3 to
6bcaea9
Compare
…ion access Removed version_view as it was redundant with meta_view. Added nano::store::get_version / set_version wrappers delegating to backend version access (via meta_view) and updated all call sites.
Decouple generic meta access from versioning and update ledger store and backend accordingly. Prepares backend for additional stores with independent versions.
This refactors selected store mutations to go through nano::ledger methods instead of accessing stores directly. This enables future maintenance of extended index tables behind an extended_ledger flag without spreading index logic across the codebase.
This change introduces an optional ext_ledger_store with its own schema, versioning, and upgrade logic.
…dicated nano::ext_ledger Keeps data processing and index population out of the store layer. The ext_ledger_store is now limited to schema management and version/table migrations via perform_upgrades. This establishes a clearer separation of responsibilities between storage mechanics and ledger-level logic.
… being disabled in config
The index is automatically backfilled from existing ledger data on initialization and kept consistent via ledger put/delete block hooks.
The index is automatically backfilled from existing ledger data on initialization and kept consistent via ledger update account hooks.
…en available Use the account_delegators_by_weight extended ledger index to serve delegators and delegators_count RPCs when the extended ledger is available.
The index is automatically backfilled from existing pending entries on initialization and kept consistent via ledger put/delete receivable hooks.
Use the account_receivables_by_amount extended ledger index to serve receivable RPC when the extended ledger is available.
6bcaea9 to
77ba2d7
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request introduces an optional extended ledger store with initial index implementations.
This is a work in progress - feedback is very welcome! 😄
ToDo: