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

Full DBM propagation for pgx #2711

Open
ewhauser opened this issue May 25, 2024 · 8 comments
Open

Full DBM propagation for pgx #2711

ewhauser opened this issue May 25, 2024 · 8 comments
Assignees
Labels
apm:ecosystem contrib/* related feature requests or bugs proposal/accepted Accepted proposals proposal more in depth change that requires full team approval

Comments

@ewhauser
Copy link

ewhauser commented May 25, 2024

pgx needs support for DBM trace propagation (https://docs.datadoghq.com/database_monitoring/connect_dbm_and_apm/?tab=go) like lib/pq

@ewhauser ewhauser added the enhancement quick change/addition that does not need full team approval label May 25, 2024
@github-actions github-actions bot added the needs-triage New issues that have not yet been triaged label May 25, 2024
@felixge felixge removed the needs-triage New issues that have not yet been triaged label Jun 6, 2024
@felixge
Copy link
Member

felixge commented Jun 6, 2024

👋 Thanks for submitting this and sorry for the delay. I've pinged the internal team owning this, hopefully they'll get to it ASAP.

@rarguelloF
Copy link
Contributor

Hello @ewhauser, thanks for opening an issue!

What instrumentation are you referring to? We currently don't have any instrumentation named dbx.

If your issue is about supporting a new instrumentation, please feel free to open a new issue with more details about your request, including a link to the github repository of the library.

We accept external PRs from external contributors, but if you want us to tackle the work feel free to reach out Datadog Support so we can best prioritize that work!

@rarguelloF rarguelloF added waiting-for-info waiting for answer from issue creator proposal more in depth change that requires full team approval and removed enhancement quick change/addition that does not need full team approval labels Jun 6, 2024
@rarguelloF rarguelloF self-assigned this Jun 6, 2024
@ewhauser ewhauser changed the title Full DBM propagation for dbx Full DBM propagation for pgx Jun 6, 2024
@ewhauser
Copy link
Author

ewhauser commented Jun 6, 2024

Apologies, that was I typo. I updated it to be pgx.

@caseycull
Copy link

caseycull commented Jun 6, 2024

Hi @ewhauser, which DBMS types would you want this for?
(Postgres, MySQL, SQL Server, Oracle)

@ewhauser
Copy link
Author

ewhauser commented Jun 6, 2024

Postgres

@rarguelloF rarguelloF added proposal/accepted Accepted proposals apm:ecosystem contrib/* related feature requests or bugs and removed waiting-for-info waiting for answer from issue creator labels Jun 7, 2024
@doron-cohen
Copy link
Contributor

Hey, correlating queries to traces and seeing calling services in the advanced DB monitoring in Datadog could be really beneficial to us. Is there a plan to support this? Is there a custom way to add a comment to the query so it will be picked up? Thanks

@fcarrero
Copy link

any update about it, it is a great feature for us

@callmemc
Copy link

callmemc commented Sep 4, 2024

Hi @caseycull and @rarguelloF, thanks for getting the ball rolling on this! Any updates you can share? Would also find this really useful. We use Postgres

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
apm:ecosystem contrib/* related feature requests or bugs proposal/accepted Accepted proposals proposal more in depth change that requires full team approval
Projects
None yet
Development

No branches or pull requests

7 participants