-
Notifications
You must be signed in to change notification settings - Fork 2.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
Support reading multiple trace IDs from archive reader #956
Conversation
} | ||
if aH.handleError(w, err, http.StatusInternalServerError) { | ||
return | ||
} | ||
process(trace) | ||
} | ||
|
||
func 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.
every combination of getTrace
, getTraceFromReaders
, etc. was taken. Uh getTraceFromReadersInternal
?
Codecov Report
@@ Coverage Diff @@
## master #956 +/- ##
=====================================
Coverage 100% 100%
=====================================
Files 136 136
Lines 6320 6322 +2
=====================================
+ Hits 6320 6322 +2
Continue to review full report at Codecov.
|
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 suggest adding a test.
…Ds are passed as parameters Signed-off-by: Won Jun Jang <wjang@uber.com>
1c7bff3
to
0686045
Compare
…Ds are passed as parameters
Signed-off-by: Won Jun Jang wjang@uber.com
Which problem is this PR solving?