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

debt: remove unmaintained tracing code #2158

Closed
mastercactapus opened this issue Feb 14, 2022 · 2 comments · Fixed by #2420
Closed

debt: remove unmaintained tracing code #2158

mastercactapus opened this issue Feb 14, 2022 · 2 comments · Fixed by #2420

Comments

@mastercactapus
Copy link
Member

Tracing code was added with spans for debugging things like GraphQL & DB calls that we have other tools for now. This code hasn't been maintained for a long time and just adds noise, complexity, and build time.

We should pull it out and remove the dependencies:

  • cloud.google.com/go
  • contrib.go.opencensus.io/exporter/jaeger
  • contrib.go.opencensus.io/exporter/stackdriver
  • go.opencensus.io

If requested we can re-introduce it in the future in a planned and maintainable way.

@alishahwee
Copy link

Hey, could I take this on?

@mastercactapus
Copy link
Member Author

@alishahwee Absolutely! It will touch a lot of files, so feel free to break it into pieces/multiple PRs if you run into tricky bits.

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 a pull request may close this issue.

2 participants