-
Notifications
You must be signed in to change notification settings - Fork 4
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
feat(receiver-mock): add querying the spans and traces by span name #325
feat(receiver-mock): add querying the spans and traces by span name #325
Conversation
98e1b6e
to
38a97c3
Compare
src/rust/receiver-mock/README.md
Outdated
@@ -47,7 +47,7 @@ The following endpoints provide information about received traces: | |||
It accepts a list of key value pairs being an attribute set that the span will | |||
have to fullfil in order to be returned. | |||
Attribute values can be omitted in which case only presence of a particular attribute | |||
will be checked. | |||
will be checked. `__name__` is handled specially as it will be matched against the span name. |
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 would move it to next line 🤷
if span.name == *value { | ||
continue; | ||
} else { | ||
return false; | ||
} |
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.
if span.name == *value { | |
continue; | |
} else { | |
return false; | |
} | |
if span.name = *value { | |
continue; | |
} | |
return false; |
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.
Left some minor comments :)
38a97c3
to
62f85e2
Compare
62f85e2
to
e672ea4
Compare
e672ea4
to
e1cec7b
Compare
No description provided.