Skip to content

Implement batch get #251

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 8 commits into from
Nov 3, 2025
Merged

Implement batch get #251

merged 8 commits into from
Nov 3, 2025

Conversation

@carte7000
Copy link
Contributor

@carte7000 carte7000 commented Oct 31, 2025

No description provided.

@github-actions
Copy link

github-actions bot commented Nov 3, 2025

E2E Smoke Test Results

Test Case Status Duration
TestE2ESmoke/extra_args_v2/src->dst_msg_execution_eoa_receiver pass 10.72s
TestE2ESmoke/extra_args_v2/dst->src_msg_execution_eoa_receiver pass 10.22s
TestE2ESmoke/extra_args_v2/1337->3337_msg_execution_mock_receiver pass 9.72s
TestE2ESmoke/extra_args_v2 pass 30.66s
TestE2ESmoke/extra_args_v3/EOA_receiver_and_default_committee_verifier pass 9.72s
TestE2ESmoke/extra_args_v3/EOA_receiver_and_secondary_committee_verifier pass 10.22s
TestE2ESmoke/extra_args_v3/receiver_w/_secondary_verifier_required pass 10.72s
TestE2ESmoke/extra_args_v3/receiver_w/_secondary_required_and_tertiary_optional_threshold=1 pass 10.22s
TestE2ESmoke/extra_args_v3/receiver_w/_default_required,_secondary_and_tertiary_optional,_threshold=1,_message_specifies_all_three pass 9.72s
TestE2ESmoke/extra_args_v3/receiver_w/_default_required,_secondary_and_tertiary_optional,_threshold=1,_message_specifies_default_and_secondary pass 9.72s
TestE2ESmoke/extra_args_v3/receiver_w/_default_required,_secondary_and_tertiary_optional,_threshold=1,_message_specifies_default_and_tertiary pass 10.22s
TestE2ESmoke/extra_args_v3/EOA_receiver_and_default_committee_verifier#01 pass 9.72s
TestE2ESmoke/extra_args_v3/max_data_size pass 10.22s
TestE2ESmoke/extra_args_v3/EOA_receiver_and_default_committee_verifier_with_token_transfer pass 9.73s
TestE2ESmoke/extra_args_v3 pass 100.21s
TestE2ESmoke pass 131.08s

Full logs are available in the workflow artifacts.

@github-actions
Copy link

github-actions bot commented Nov 3, 2025

Code coverage report:

Package main simon/aggregator/batch-verifier-result
aggregator 48.01% 46.92%
cciptestinterfaces 100.00% 100.00%
ccv-evm 0.00% 0.00%
cmd 0.00% 0.00%
executor 36.32% 36.32%
indexer 28.81% 36.57%
integration 4.27% 4.27%
protocol 42.02% 42.02%
verifier 43.20% 43.20%

@carte7000 carte7000 marked this pull request as ready for review November 3, 2025 16:16
@carte7000 carte7000 requested review from a team and skudasov as code owners November 3, 2025 16:16
@carte7000 carte7000 merged commit 7c7ab07 into main Nov 3, 2025
11 checks passed
@carte7000 carte7000 deleted the simon/aggregator/batch-verifier-result branch November 3, 2025 16:16
0xAustinWang pushed a commit that referenced this pull request Nov 7, 2025
@carte7000 @0xAustinWang
Implement batch get (#251)
dcc21c7 
* Implement batch get

* Linting

* Linting

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

Reviewers

@asoliman92 asoliman92 asoliman92 approved these changes

@skudasov skudasov Awaiting requested review from skudasov skudasov is a code owner

@AndresJulia AndresJulia Awaiting requested review from AndresJulia

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@carte7000 @asoliman92