mdebray/tvl-depot
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

docs(tvix/todo): add section on [ca]store errors

Browse source 
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>
This commit is contained in:
Florian Klink 2024-05-10 09:14:09 +03:00 committed by clbot
parent fd49dfa6ca
commit 728ceda8a6
1 changed files with 7 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
tvix/docs/src/TODO.md
View file

@ -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 with a different level of `--strict`, but the toplevel doc-comment suggests
its generic? 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 ## Fixes towards correctness
- `builtins.toXML` is missing string context. See b/398. - `builtins.toXML` is missing string context. See b/398.
- `builtins.toXML` self-closing tags need to be configurable in a more granular - `builtins.toXML` self-closing tags need to be configurable in a more granular