Skip to content

[DAP-18] Remove PartialBatchSelector from CollectionJobResp - #4818

Merged
jcjones merged 2 commits into
mainfrom
4783-remove-partial-batch-selector
Aug 20, 2026
Merged

[DAP-18] Remove PartialBatchSelector from CollectionJobResp#4818
jcjones merged 2 commits into
mainfrom
4783-remove-partial-batch-selector

Conversation

@jcjones

@jcjones jcjones commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

The DAP spec dropped the field (ietf-wg-ppm/draft-ietf-ppm-dap#778); the response is now {report_count, interval, leader_encrypted_agg_share, helper_encrypted_agg_share}. Cascades into the collector's public Collection type, the collect CLI's batch-id output, and the interop collector, which can no longer observe a leader-selected batch ID from the response.

Fixes #4783

The DAP spec dropped the field (draft-ietf-ppm-dap#778); the response
is now {report_count, interval, leader_encrypted_agg_share,
helper_encrypted_agg_share}. Cascades into the collector's public
Collection type, the collect CLI's batch-id output, and the interop
collector, which can no longer observe a leader-selected batch ID from
the response.

Fixes #4783
@jcjones jcjones changed the title Remove PartialBatchSelector from CollectionJobResp [DAP-18] Remove PartialBatchSelector from CollectionJobResp Aug 19, 2026
Comment thread messages/src/lib.rs
Comment thread collector/src/lib.rs
@jcjones
jcjones marked this pull request as ready for review August 20, 2026 17:57
@jcjones
jcjones requested a review from a team as a code owner August 20, 2026 17:57
@jcjones
jcjones merged commit a7feb91 into main Aug 20, 2026
8 checks passed
@jcjones
jcjones deleted the 4783-remove-partial-batch-selector branch August 20, 2026 21:43
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.

Remove PartialBatchSelector from CollectionJobResp

3 participants