diff options
Diffstat (limited to 'tvix/docs/src/TODO.md')
| -rw-r--r-- | tvix/docs/src/TODO.md | 14 |
1 files changed, 14 insertions, 0 deletions
diff --git a/tvix/docs/src/TODO.md b/tvix/docs/src/TODO.md index e3f867e11ac2..3d2e083ef9f9 100644 --- a/tvix/docs/src/TODO.md +++ b/tvix/docs/src/TODO.md @@ -77,6 +77,20 @@ correctness: "some amount of outgoing bytes" in memory. This is somewhat blocked until the {Chunk/Blob}Service split is done, as then prefetching would only be a matter of adding it into the one `BlobReader`. + - The import builtins (`builtins.path` and `builtins.filterSource`) use(d) some + helper functions in TvixStoreIO that deals with constructing the `PathInfo` + structs and inserting them, but some of the abstractions where probably done + at the wrong layer: + - Some ways of importing calculate the NAR representation twice + - The code isn't very usable from other consumers that also create structs + `PathInfo`. + - `node_to_path_info` is ony called by `register_in_path_info_service` (due + to this marked as private for now). + Instead of fighting these abstractions, maybe it's best to explicitly add the + code back to the two builtins, remove redundant calls and calculations. A + later phase could then see how/if some of this can be reasonably factored out in + a way it's also usable for other places having a node and wanting to persist + it (if at all). ### Error cleanup - Currently, all services use tvix_castore::Error, which only has two kinds |