Add a comment explaining BA32 usage in playbook

akoshelev · Jun 14, 2024 · 00376d0 · 00376d0
1 parent 0e3bf1a
commit 00376d0
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/ipa-core/src/bin/report_collector.rs b/ipa-core/src/bin/report_collector.rs
@@ -256,6 +256,9 @@ async fn ipa(
     let mut key_registries = KeyRegistries::default();
     let actual = match query_style {
         IpaQueryStyle::Oprf => {
+            // the value for histogram values (BA32) must be kept in sync with the server-side
+            // implementation, otherwise a runtime reconstruct error will be generated.
+            // see ipa-core/src/query/executor.rs
             playbook_oprf_ipa::<BA32, _>(
                 input_rows,
                 helper_clients,