[Tarantool-patches] [PATCH] box: rfc for stacked diagnostic area
Nikita Pettik
korablev at tarantool.org
Wed Jan 29 17:36:06 MSK 2020
On 28 Jan 01:23, Vladislav Shpilevoy wrote:
> Hi! Thanks for the fixes!
>
> It is not a second review. I just wanted to say that I am
> going to spend less time on reviewes to the end of this
> week due to intensive preparations to FOSDEM. I will get
> back to this RFC as soon as I can, I didn't forget.
>
> Just one comment for now, below.
>
> >>> +* **Issues**: [#1148](https://github.com/tarantool/<repository\>/issues/1148)
> >>> +
> >>
> >> 2. You missed 'Summary' section.
> >
> > I believe it is redundant for such brief RFC.
> >
>
> Nevertheless, please, add it. If that RFC is so simpler,
> then it should not be a problem to make it match the
> RFC template. Template says:
>
> "Short description what, why and how is implemented."
>
> So just say something like
>
> The document describes a stacked diagnostics feature. The
> feature is needed for the cases, when there is a complex and
> big callstack with different subsystems in it, and sometimes
> the most lowlevel error is not descriptive enough. So it is
> wanted to be able to look at errors along the whole callstack
> from the place where a first error happened. The feature
> implementation is going to change a fiber's error object to a
> list of objects, a stack. Its first element will always be
> created by the deepest and the most basic errors.
>
> Or anything you think would be better here.
Ok, as you wish. A bit reformulated your description:
diff --git a/doc/rfc/1148-stacked-diagnostics.md b/doc/rfc/1148-stacked-diagnostics.md
index 0d1d07dc3..d57e040ba 100644
--- a/doc/rfc/1148-stacked-diagnostics.md
+++ b/doc/rfc/1148-stacked-diagnostics.md
@@ -6,6 +6,18 @@
Pettik Nikita @Korablev77 korablev at tarantool.org
* **Issues**: [#1148](https://github.com/tarantool/<repository\>/issues/1148)
+
+## Summary
+
+The document describes a stacked diagnostics feature. It is needed for the cases,
+when there is a complex and huge call stack with variety of subsystems in it.
+It may turn out that the most low-level error is not descriptive enough. In turn
+user may want to be able to look at errors along the whole call stack from the
+place where a first error happened. In terms of implementation single fiber's
+error object is going to be replaced with a list of objects forming diagnostic
+stack. Its first element will always be created by the deepest and the most
+basic error. Both C and Lua APIs are extended to support adding errors in stack.
+
More information about the Tarantool-patches
mailing list