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

[Plugin]: Add postgres plugin #404

Closed
markwolff opened this issue Oct 4, 2019 · 2 comments · Fixed by #417
Closed

[Plugin]: Add postgres plugin #404

markwolff opened this issue Oct 4, 2019 · 2 comments · Fixed by #417
Assignees
Milestone

Comments

@markwolff
Copy link
Member

Describe the solution you'd like
Add plugin similar to mongodb plugin (#205) for the node.js postgres driver

Additional context
This can be assigned to me if this is a plugin that this project would like to have.

@mayurkale22
Copy link
Member

Thanks for opening the issue.

+1 to add this plugin. The (pg) module seems to have enough downloads (4m/month) on npm to mean that there are a lot users - Substantial use.

@open-telemetry/node-approvers and @open-telemetry/node-maintainers WDYT?

@mayurkale22 mayurkale22 added this to the Basic Plugins milestone Oct 4, 2019
@vmarchaud
Copy link
Member

vmarchaud commented Oct 4, 2019

Sure, you can find a opencensus implementation there that i've done at my previous company: https://github.com/keymetrics/pm2-io-apm/blob/master/src/census/plugins/pg.ts

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants