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.

Sign up for GitHub

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

Jump to bottom

enable datastore queries at debug level for all datastores #1055

Closed
vijayaraj opened this issue Dec 15, 2022 · 3 comments
Closed

enable datastore queries at debug level for all datastores #1055

vijayaraj opened this issue Dec 15, 2022 · 3 comments
Labels
area/datastore Affects the storage system area/observability Affects telemetry data good first issue hint/good first issue Someone new could handle this

Comments

@vijayaraj
Copy link

Currently, there is no way to check the queries sent to the datastore when making API calls. It would be useful if we can see them as part of the logs.
@vroldanbet
Copy link
Contributor

vroldanbet commented Dec 15, 2022 •
edited
Loading

PostgreSQL and CockroachDB currently show SQL queries at debug level. We'd have to check the other datastores to determine if it's supported.

@vroldanbet vroldanbet changed the title Add datastore queries to logs enable datastore queries at debug level for all datastores Dec 15, 2022
@vroldanbet vroldanbet added area/datastore Affects the storage system area/observability Affects telemetry data labels Dec 15, 2022
@vroldanbet vroldanbet added hint/good first issue Someone new could handle this good first issue labels Jul 5, 2024
@josephschorr
Copy link
Member

Spanner support was added in #2181

barakmich added a commit that referenced this issue Jan 29, 2025
@barakmich
[mysql] pipe logger into mysql for debug printing (#1055)
1637f0b
barakmich added a commit that referenced this issue Jan 29, 2025
@barakmich
[mysql] pipe logger into mysql for debug printing (#1055)
fa5e56e
github-merge-queue bot pushed a commit that referenced this issue Jan 29, 2025
@barakmich
Merge pull request #2217 from authzed/mysql_logging
542053f 
[mysql] pipe logger into mysql for debug printing (#1055)
@barakmich
Copy link
Contributor

Closing because I think MySQL was the only outlier!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area/datastore Affects the storage system area/observability Affects telemetry data good first issue hint/good first issue Someone new could handle this
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@barakmich @vijayaraj @josephschorr @vroldanbet