Skip to content

feat: use generic key value config for prover_db_indexing - #209

Merged
JayWhite2357 merged 1 commit into
mainfrom
feat/use-generic-config-for-prover-db
Jul 30, 2026
Merged

feat: use generic key value config for prover_db_indexing#209
JayWhite2357 merged 1 commit into
mainfrom
feat/use-generic-config-for-prover-db

Conversation

@JayWhite2357

Copy link
Copy Markdown
Contributor

Rationale for this change

#208 introduces generic key-value config for offchain workers. This PR uses it for the prover_db_indexing OCW.

What changes are included in this PR?

  • ProverDbConsumerConfig is populated from the new generic key-value config
  • max_blocks_per_invocation and ocw_lock_deadline_ms are added to the configuration so they can be chosen by a node operator rather than hardcoded by the runtime. The default values keep the old values.

Are these changes tested?

Yes

@JayWhite2357
JayWhite2357 requested review from a team as code owners July 29, 2026 16:21
@github-actions

github-actions Bot commented Jul 29, 2026

Copy link
Copy Markdown

1.70.0

Bug Fixes

  • make on chain indexer keys saturate rather than default to (5f794de
  • use finalized block in prover-db-indexer (50438ab

Features

  • add helper that queries the client for s. (4fc3952
  • add client native interface and extenality (d9c6c4d
  • add generic config key-value store for offchain workers (117b845
  • Document Snapshot Download Process (4c2608b
  • register client externality to offchain worker (338c213
  • use generic key value config for prover_db_indexing instead of storage/CLI based config (faff564

@JayWhite2357
JayWhite2357 requested a review from tlovell-sxt July 29, 2026 16:24
@JayWhite2357
JayWhite2357 force-pushed the feat/use-generic-config-for-prover-db branch from 81867d8 to 154fa79 Compare July 29, 2026 16:28
tlovell-sxt
tlovell-sxt previously approved these changes Jul 29, 2026

@tlovell-sxt tlovell-sxt left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good

@JayWhite2357
JayWhite2357 force-pushed the feat/use-generic-config-for-prover-db branch from 154fa79 to 78927f9 Compare July 30, 2026 13:22
@JayWhite2357
JayWhite2357 requested a review from tlovell-sxt July 30, 2026 13:28
@JayWhite2357
JayWhite2357 force-pushed the feat/use-generic-config-for-prover-db branch 4 times, most recently from 1699a2f to 88e37a3 Compare July 30, 2026 18:05
tlovell-sxt
tlovell-sxt previously approved these changes Jul 30, 2026
@JayWhite2357
JayWhite2357 merged commit 21a49d5 into main Jul 30, 2026
9 checks passed
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