-
Notifications
You must be signed in to change notification settings - Fork 724
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
trace-dispatcher: documentation #5635
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I found the explanation of the tracing system enlightening. It's great to have more code docs!
31ee69c
to
64158f8
Compare
|
||
### Privacy | ||
documentFor :: Namespace a -> Maybe Text |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No default implementation like const Nothing
? Do I have to provide that explicitly as a dev?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Has currently no default implementation. The idea was to motivate the writing of docu, but this can be changed?
|
||
2. __Trace-Forward Tracer:__ This is a network-only sink dedicated to forwarding messages using typed protocols over TCP or local sockets. It exclusively handles public traces and should be instantiated only once per application. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is forwarding over TCP an option, or has that been removed? A quick look of how I'd configure this came up empty...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can't see the relation of your question to this part. The configuration of the forwarder should be part of the other two documents: "cardano-tracer manual" and "quick start of new tracing for admins"
ddd635a
to
99e94d4
Compare
99e94d4
to
dca4b8c
Compare
576ab7c
to
8afc2da
Compare
8afc2da
to
342933a
Compare
Complete rewrite of trace-dispatcher documentation.