diff options
| author | Florian Klink <flokli@flokli.de> | 2024-05-10T06·14+0300 |
|---|---|---|
| committer | clbot <clbot@tvl.fyi> | 2024-05-11T14·59+0000 |
| commit | 728ceda8a6a2fd74d9be5739f7c47e3eabb36ff8 (patch) | |
| tree | 42bfc6a9233409cb155a240b8dfc0b786f0018fc | |
| parent | fd49dfa6ca9db0f3a026a581ae2a14f760ac251d (diff) | |
docs(tvix/todo): add section on [ca]store errors r/8110
Change-Id: I832eed8abb62b7e13972ed58649a96e31b17f033 Reviewed-on: https://cl.tvl.fyi/c/depot/+/11610 Tested-by: BuildkiteCI Autosubmit: flokli <flokli@flokli.de> Reviewed-by: Connor Brewster <cbrewster@hey.com>
| -rw-r--r-- | tvix/docs/src/TODO.md | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/tvix/docs/src/TODO.md b/tvix/docs/src/TODO.md index 50b41e3e50bd..f06a9b852da4 100644 --- a/tvix/docs/src/TODO.md +++ b/tvix/docs/src/TODO.md @@ -25,6 +25,13 @@ sure noone is working on this, or has some specific design in mind already. with a different level of `--strict`, but the toplevel doc-comment suggests its generic? +### Error cleanup + - Currently, all services use tvix_castore::Error, which only has two kinds + (invalid request, storage error), containing an (owned) string. + This is quite primitive. We should have individual error types for BS, DS, PS. + Maybe these should have some generics to still be able to carry errors from + the underlying backend, similar to `IngestionError`. + ## Fixes towards correctness - `builtins.toXML` is missing string context. See b/398. - `builtins.toXML` self-closing tags need to be configurable in a more granular |