Skip to content
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

Design doc #188

Closed
wants to merge 3 commits into from
Closed

Conversation

connorlindsey
Copy link
Contributor

The attached design document outlines our plan to implement a Cortex Exporter for the Go SDK. This shouldn't be merged, but is instead a request for review before we begin implementing the Exporter. Please leave comments below or in the document. Thank you!

cc @ercl @alolita

@MrAlias
Copy link
Contributor

MrAlias commented Aug 13, 2020

@connorlindsey I'd say this is "Accepted", but not sure if that means it can be closed? I'll leave it up to you.

plantfansam referenced this pull request in plantfansam/opentelemetry-go-contrib Mar 18, 2022
Since these methods are about the span itself rather than specifically
*events* on the span, it makes sense to drop "events" from their names.

[Closes #33]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants