-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
fix: time range from traces to logs #6634
Conversation
Build Error! No Linked Issue found. Please link an issue or mention it in the body using #<issue_id> |
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.
👍 Looks good to me! Reviewed everything up to 4693660 in 11 seconds
More details
- Looked at
17
lines of code in1
files - Skipped
0
files when reviewing. - Skipped posting
1
drafted comments based on config settings.
1. frontend/src/container/TraceDetail/SelectedSpanDetails/index.tsx:90
- Draft comment:
Ensure thattraceStartTime
andtraceEndTime
are in milliseconds before subtracting or adding 1000. If they are in nanoseconds, the adjustment should be 1,000,000 instead. - Reason this comment was not posted:
Comment did not seem useful.
Workflow ID: wflow_NExnmT3QxpizTx9H
You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet
mode, and more.
Build Error! No Linked Issue found. Please link an issue or mention it in the body using #<issue_id> |
contributes to - #6124 |
Summary
contributes to - #6124