From 728ceda8a6a2fd74d9be5739f7c47e3eabb36ff8 Mon Sep 17 00:00:00 2001 From: Florian Klink Date: Fri, 10 May 2024 09:14:09 +0300 Subject: docs(tvix/todo): add section on [ca]store errors Change-Id: I832eed8abb62b7e13972ed58649a96e31b17f033 Reviewed-on: https://cl.tvl.fyi/c/depot/+/11610 Tested-by: BuildkiteCI Autosubmit: flokli Reviewed-by: Connor Brewster --- tvix/docs/src/TODO.md | 7 +++++++ 1 file changed, 7 insertions(+) (limited to 'tvix/docs/src') diff --git a/tvix/docs/src/TODO.md b/tvix/docs/src/TODO.md index 50b41e3e50..f06a9b852d 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 -- cgit 1.4.1