sapling/eden/mononoke/derived_data
Thomas Orozco 341b4f1bc3 mononoke/filenodes: expect a Vec of filenodes to insert
Summary:
The API expects a stream of filenodes to insert, but we actually never used
that ability. Instead, every single callsites has a `Vec`, which it converts to
a stream and passes that in.

I'd like to change this for two reasons:

- It's un-necessary
- It makes the code more complex on the Filenodes implementation side, and less
  efficient, since we need to `chunk()` there in small chunks, which might not
  all be in the same shard. If we get the entire `Vec` at once, we can chunk on a
  per-shard basis (this happens later in this stack).

Besides, if we end up having a stream and wanting the old behavior, we can
always call `chunk()` the stream and call `add_filenodes` on each batch (which
is actually nicer because if you have a futures 0.2 stream that isn't static,
you can do this, but you can't turn it into a `BoxStream`!).

Reviewed By: StanislavGlebik

Differential Revision: D19902537

fbshipit-source-id: a4c030c4a51afbb6e9db133b32464009eed197af
2020-02-27 12:34:18 -08:00
..
blame derived_data: add DeriveError for when derivation is disabled 2020-02-19 09:28:09 -08:00
deleted_files_manifest mononoke: update internals to Bytes 0.5 2020-02-27 08:08:28 -08:00
fastlog mononoke/scs: use last change in file history 2020-02-26 08:09:12 -08:00
filenodes mononoke/filenodes: expect a Vec of filenodes to insert 2020-02-27 12:34:18 -08:00
fsnodes mononoke: update internals to Bytes 0.5 2020-02-27 08:08:28 -08:00
mercurial_derived_data remove the need to pass mapping to ::derive method 2020-02-12 10:22:39 -08:00
src mononoke: asyncify derived data 2020-02-27 12:17:54 -08:00
unodes mononoke: update internals to Bytes 0.5 2020-02-27 08:08:28 -08:00
utils derived_data: add DeriveError for when derivation is disabled 2020-02-19 09:28:09 -08:00