about summary refs log tree commit diff
diff options
context:
space:
mode:
authorFlorian Klink <flokli@flokli.de>2024-05-10T06·14+0300
committerclbot <clbot@tvl.fyi>2024-05-11T14·59+0000
commit728ceda8a6a2fd74d9be5739f7c47e3eabb36ff8 (patch)
tree42bfc6a9233409cb155a240b8dfc0b786f0018fc
parentfd49dfa6ca9db0f3a026a581ae2a14f760ac251d (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.md7
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