Skip to content
This repository has been archived by the owner on Jul 21, 2023. It is now read-only.

facilitator: generate sample ingestion data #1

Closed
wants to merge 4 commits into from
Closed

Conversation

tgeoghegan
Copy link
Collaborator

This PR adds a Rust target for the facilitator binary that will implement the map and reduce jobs for the facilitator server. At the moment, it only implements generation of sample ingestion data from random values.

This commit adds the start of the facilitator Rust crate, which will be
the executable that processes data shares, does validation and emits
aggregates. This commit includes support for reading ingestion headers,
signatures and data share packets in and out of the Avro format.
Starting toward sample ingestion data generation
Implements all(?) the necessary command line flags and writes an Avro
ingestion header to the specified path.
sample.rs can now emit an ingestion batch header and arbitrarily many
data share packets. It emits ingestion data for both the PHA and
facilitator servers.

I tried for a long time to get a generic interface over MemoryTransport
and FileTransport working, but I can't quite get the generic put() on
MemoryTransport to work because of issues with the lifetime of the
boxed object. I have given up for now and just made tests use temporary
dirs that get cleaned up automatically (testing was the only point of
MemoryTransport to begin with).
@tgeoghegan
Copy link
Collaborator Author

I have made more progress beyond what's in this PR, including substantial refactorings of the code in here, in the timg/ingestion-validation branch. To avoid redundant reviews and rebases, I'm going to rescind this PR and work with yuriks to review a larger changeset.

@tgeoghegan tgeoghegan closed this Sep 19, 2020
@tgeoghegan tgeoghegan deleted the timg/avro branch September 29, 2020 20:19
aaomidi pushed a commit that referenced this pull request Feb 2, 2021
aaomidi pushed a commit that referenced this pull request Feb 3, 2021
* Explore upgrading to ureq 2

* Feedback #1

* Feedback #2

Co-authored-by: Tim Geoghegan <timg@letsencrypt.org>

* Feedback #3

Co-authored-by: Tim Geoghegan <timg@letsencrypt.org>
tgeoghegan added a commit that referenced this pull request Jun 25, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant