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

feat!: upgrade to iota_sdk:2.0 #1297

Draft
wants to merge 77 commits into
base: main
Choose a base branch
from
Draft

feat!: upgrade to iota_sdk:2.0 #1297

wants to merge 77 commits into from

Conversation

DaughterOfMars
Copy link
Collaborator

@DaughterOfMars DaughterOfMars commented Nov 8, 2023

Description

This PR upgrades to the iota_sdk 2.0 version and refactors a lot of the architecture of chronicle in keeping with that.

Implementation Details

Some important changes that have occurred:

  1. Most custom model types have been removed in favor of storing the serialized bytes of Output and SignedBlock. In order to index these collections, fields have been added at the top level for the specific indexed fields only.
  2. Milestones have, more-or-less, been wholesale replaced by Slots. Instead of a MilestoneCollection there is a CommittedSlotCollection, which is used to track the progress of syncing.

Related Issues

#1315

Notes to Reviewer

As a reviewer, please pay particular attention to the following areas when reviewing this PR and tick the above boxes after you have completed the steps.

Config Changes

  • Ensure proper order in which CLI and config arguments are applied.
  • Ensure that config changes work with individual build features by running cargo ci-check-features.

API Changes

  • Test the API endpoints which were added/changed.
  • Ensure that the API response times scale with database size appropriately.
  • Review the API documentation changes and confirm that it matches the actual functionality.
  • Check for breaking changes in the API and matching (conventional) commit message prefix.

Test cases

  • Review and run tests that were added/changed.
  • Suggest places that may benefit from test cases.

INX Changes

  • Run chronicle using an INX connection.

Database Changes

  • Review database queries for correctness/conciseness.
  • Ensure queries are supported by indexes if needed.
  • Check for breaking changes in the data model and matching (conventional) commit message prefix.

Alex Coats and others added 30 commits February 28, 2024 12:16
…outes. Add count to blocks by slot/commitment.
…ytics for richest addresses and token distribution endpoints
…ger update store (#1381)

* analytics: ensure finalized transactions

* suggestions
…nalized blocks (#1385)

* filter out non-finalized blocks

* remove block state analytics

* update dashboard

* much better

---------

Co-authored-by: DaughterOfMars <alexandcoats@gmail.com>
chore: release nova v1.0.0-beta.1
* add check

* nit

* review
Wait for slots to be finalized in INX
chore: release nova-v1.0.0-beta.2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants