search

ietf-wg-ppm / draft-ietf-ppm-dap

This document describes the Distributed Aggregation Protocol (DAP) being developed by the PPM working group at IETF.
Other
46 stars 22 forks source link

Specify "streaming" aggregation as the norm. #623

Closed branlwyd closed 1 month ago

branlwyd commented 1 month ago

Previously, aggregation was specified as if all output shares were aggregated into an aggregate share in one fell swoop, at time of collection. However, all implementations instead used an incremental aggregation process to keep a "running total" aggregate share which was updated as aggregation jobs completed, for performance reasons. This commit switches the specification to match the implementations.

Phrasing things this way also makes the name "aggregation job" make sense; as previously specified, there was no aggregation occurring inside of an aggregation job!

branlwyd commented 1 month ago

Closes #581. Closes #385.

branlwyd commented 1 month ago

(rebased on current main)

cjpatton commented 1 month ago

interim 2024/10/24: @branlwyd this is good to merge once you've addressed the last comments.