Skip to content

Conversation

@Jille
Copy link
Contributor

@Jille Jille commented Aug 14, 2023

Multiple people have asked for more involved features like stats, but with this simple context injection people can implement their own DBTX and do whatever they want from there.

My use case is also measuring query durations per query

Multiple people have asked for more involved features like stats, but
with this simple context injection people can implement their own DBTX
and do whatever they want from there.

My use case is also measuring query durations per query
@kyleconroy
Copy link
Collaborator

It's an interesting approach. Before merging this, I'd like to spend a bit more time thinking through a solution that uses the interceptor pattern from gRPC. I'm not sure it will map cleanly to sqlc-generated code, but it would solve many of the customization issues that we face today without requiring new configuration parameters.

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

Labels

None yet

2 participants