Skip to content

Conversation

@ninegua
Copy link
Member

@ninegua ninegua commented Dec 28, 2025

DEFI-1816: Print search index optimization status when starting up Rosetta

@github-actions github-actions bot added the feat label Dec 28, 2025
@ninegua ninegua marked this pull request as ready for review December 29, 2025 07:19
@ninegua ninegua requested review from a team as code owners December 29, 2025 07:19
@ninegua ninegua requested a review from mbjorkqvist December 29, 2025 07:20
);
last_log = now;
}
true // continue
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
true // continue
false // continue

I believe this should be false to continue (true -> interrupt/abort). I tried first creating a database without indexes using the following:

bazel run //rs/rosetta-api/icp:ic-rosetta-api -- --store-location /tmp --environment production --expose-metrics --log-level DEBUG --canister-id xafvr-biaaa-aaaai-aql5q-cai --token-symbol TESTICP

Then I exited Rosetta and reran the above command with the additional flag --optimize-search-indexes, which caused Rosetta to exit with the following error:

thread 'main' panicked at rs/rosetta-api/icp/src/main.rs:395:35:
Failed to initialize ledger client: InternalError(false, Details { error_message: Some("Failed to initialize SQLite database: interrupted"), extra_fields: {} })

When changing the above true to false, the new functionality in this PR was running as expected.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants