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

feat(client): DataProxy query logs #16748

Merged
merged 2 commits into from
Feb 2, 2023
Merged

feat(client): DataProxy query logs #16748

merged 2 commits into from
Feb 2, 2023

Conversation

danstarns
Copy link
Contributor

@danstarns danstarns commented Dec 12, 2022

@danstarns danstarns requested review from a team and aqrln and removed request for a team December 12, 2022 11:46
@danstarns
Copy link
Contributor Author

@SevInf SevInf force-pushed the feat/dataproxy-query-logs branch from d7823fa to 8e7e38e Compare January 31, 2023 10:28
@miguelff miguelff force-pushed the feat/dataproxy-query-logs branch 2 times, most recently from fea8723 to 0265d45 Compare February 1, 2023 11:12
@miguelff
Copy link
Contributor

miguelff commented Feb 1, 2023

I added a test for non-interactive transaction logging. I think we should merge this.

@miguelff miguelff force-pushed the feat/dataproxy-query-logs branch from 0265d45 to 366d178 Compare February 2, 2023 10:11
@miguelff miguelff requested a review from aqrln February 2, 2023 11:16
@miguelff miguelff force-pushed the feat/dataproxy-query-logs branch from f10e3fd to cb2f326 Compare February 2, 2023 11:51
@miguelff miguelff requested a review from aqrln February 2, 2023 11:53
@miguelff miguelff force-pushed the feat/dataproxy-query-logs branch from cb2f326 to 159e003 Compare February 2, 2023 11:54
@miguelff miguelff force-pushed the feat/dataproxy-query-logs branch from 159e003 to 6a8f1e5 Compare February 2, 2023 12:30
@miguelff miguelff merged commit 2df931d into main Feb 2, 2023
@miguelff miguelff deleted the feat/dataproxy-query-logs branch February 2, 2023 13:58
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