Skip to content

refactor(pxe): use findLeavesIndexes for read request verification#23123

Open
nchamo wants to merge 3 commits intomerge-train/fairiesfrom
nchamo/perf-batch-read-request-verification
Open

refactor(pxe): use findLeavesIndexes for read request verification#23123
nchamo wants to merge 3 commits intomerge-train/fairiesfrom
nchamo/perf-batch-read-request-verification

Conversation

@nchamo
Copy link
Copy Markdown
Contributor

@nchamo nchamo commented May 9, 2026

Summary

  • Replace per-read-request getNoteHashMembershipWitness / getNullifierMembershipWitness calls with a single batched findLeavesIndexes call per tree when verifying settled read requests in verifyReadRequests.
  • We only need existence here, not the witness, so the cheaper batched lookup suffices.

@nchamo nchamo added the ci-draft Run CI on draft PRs. label May 9, 2026
@nchamo nchamo self-assigned this May 9, 2026
@nchamo nchamo changed the title perf(pxe): use findLeavesIndexes for read request verification refactor(pxe): use findLeavesIndexes for read request verification May 9, 2026
@nchamo nchamo added the ci-no-fail-fast Sets NO_FAIL_FAST in the CI so the run is not aborted on the first failure label May 9, 2026
@AztecBot
Copy link
Copy Markdown
Collaborator

AztecBot commented May 9, 2026

Flakey Tests

🤖 says: This CI run detected 1 tests that failed, but were tolerated due to a .test_patterns.yml entry.

\033FLAKED\033 (8;;http://ci.aztec-labs.com/88a47d500102f2c2�88a47d500102f2c28;;�):  yarn-project/end-to-end/scripts/run_test.sh simple src/e2e_epochs/epochs_missed_l1_publish.test.ts (153s) (code: 0) group:e2e-p2p-epoch-flakes

@nchamo nchamo marked this pull request as ready for review May 9, 2026 21:50
@nchamo nchamo requested a review from benesjan May 9, 2026 23:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-draft Run CI on draft PRs. ci-no-fail-fast Sets NO_FAIL_FAST in the CI so the run is not aborted on the first failure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants