Skip to content

feat: reduce data queried by 16x#1367

Merged
ozgb merged 8 commits into
mainfrom
ozgb-reduce-cnight-query-capacity
May 5, 2026
Merged

feat: reduce data queried by 16x#1367
ozgb merged 8 commits into
mainfrom
ozgb-reduce-cnight-query-capacity

Conversation

@ozgb

@ozgb ozgb commented Apr 20, 2026

Copy link
Copy Markdown
Contributor

Overview

We do this by reducing our overestimate factor by 16x - as long as our total capacity per block is > the max utxos we can expect in a single tx, we won't get stuck syncing

🗹 TODO before merging

  • Ready
  • Verify the capacity units (tx vs utxo) through the whole idp construction EDIT: Added comments explaining this
  • Gate on runtime version

📌 Submission Checklist

  • Changes are backward-compatible (or flagged if breaking)
  • Pull request description explains why the change is needed
  • Self-reviewed the diff
  • I have included a change file, or skipped for this reason:
  • If the changes introduce a new feature, I have bumped the node minor version
  • Update documentation (if relevant)
  • Updated AGENTS.md if build commands, architecture, or workflows changed
  • No new todos introduced

🧪 Testing Evidence

Please describe any additional testing aside from CI:

  • Additional tests are provided (if possible)

🔱 Fork Strategy

  • Node Runtime Update
  • Node Client Update
  • Other:
  • N/A

Links

We do this by reducing our overestimate factor by 16x - as long as our total capacity per block is > the max utxos we can expect in a single tx, we won't get stuck syncing

Signed-off-by: Oscar Bailey <79094698+ozgb@users.noreply.github.com>
Signed-off-by: Giles Cope <gilescope@gmail.com>
@gilescope gilescope marked this pull request as ready for review April 24, 2026 16:24
@gilescope gilescope requested a review from a team as a code owner April 24, 2026 16:24
@gilescope gilescope enabled auto-merge April 24, 2026 16:24
@gilescope gilescope mentioned this pull request Apr 27, 2026
15 tasks
ozgb added 4 commits May 5, 2026 09:55
…ery-capacity

Signed-off-by: Oscar Bailey <79094698+ozgb@users.noreply.github.com>
Signed-off-by: Oscar Bailey <79094698+ozgb@users.noreply.github.com>
The real fix is an API breaking change - may address this in a later PR

Signed-off-by: Oscar Bailey <79094698+ozgb@users.noreply.github.com>
@ozgb

ozgb commented May 5, 2026

Copy link
Copy Markdown
Contributor Author

/bot rebuild-metadata

@github-actions

github-actions Bot commented May 5, 2026

Copy link
Copy Markdown
Contributor

✅ Metadata rebuild complete! Changes have been committed.

@ozgb ozgb closed this May 5, 2026
auto-merge was automatically disabled May 5, 2026 15:34

Pull request was closed

@ozgb ozgb reopened this May 5, 2026
@ozgb ozgb enabled auto-merge May 5, 2026 15:35
@ozgb ozgb added this pull request to the merge queue May 5, 2026
Merged via the queue into main with commit a035e65 May 5, 2026
53 checks passed
@ozgb ozgb deleted the ozgb-reduce-cnight-query-capacity branch May 5, 2026 17:19
@gilescope gilescope self-assigned this Jun 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component:node Node component

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants