-
Notifications
You must be signed in to change notification settings - Fork 45
docs: add standalone trace docs #1719
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
base: main
Are you sure you want to change the base?
Conversation
Deploying greptime-docs with
|
Latest commit: |
3e1fb22
|
Status: | ✅ Deploy successful! |
Preview URL: | https://01be728c.greptime-docs.pages.dev |
Branch Preview URL: | https://docs-trace-section.greptime-docs.pages.dev |
Translated docs coming later |
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.
Great job! Almost LGM
In this section, we will cover how trace data is modeled in GreptimeDB as | ||
tables. | ||
|
||
We reuse the concept of Pipeline for trace data modeling. However, note that at |
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.
Consider linking to the pipeline documentation: https://docs.greptime.com/user-guide/logs/manage-pipelines.
Should we move pipeline management documentation out of the logs user guide and create a standalone document, similar to the flow documentation?
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.
Here I ignored link by purpose, because we only support non-customiziable built-in pipeline.
|
||
### GreptimeDB Pipeline | ||
|
||
The HTTP header `x-greptime-pipeline-name` is required for ingesting trace |
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.
The same suggestion as above. Link the pipeline concept document here.
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.
LGTM
What's Changed in this PR
Describe the change in this PR
Add a standalone section for trace.
OpenTelemetry Traces
to this chapterChecklist
sidebars.ts
matches the current document structure when you changed the document structure.