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

Extend the listener mechanism so that you can instrument the QueryDSL execution #878

Closed
bbakerman opened this Issue Aug 3, 2014 · 2 comments

Comments

Projects
None yet
2 participants
@bbakerman
Contributor

bbakerman commented Aug 3, 2014

We would love to be able to performance instrument the QuerySQL execution and database operations.

One aspect of that is to have an extended instrumented call back mechanism as SQL gets built and executed

The current SQLListener is a step in the right direction but not extensive enough

An instrumentation listener interface needs a "setup" method to obtain a "context" object and then pass it to each listener call back step.

This allows the instrumentee to squirrel away the context it needs to record numbers.

I propose extending the current listener interface with a listener2 that has this shape

@bbakerman

This comment has been minimized.

Show comment
Hide comment
@bbakerman

bbakerman Aug 3, 2014

Contributor

I shall be submitting a PR to get this started

Contributor

bbakerman commented Aug 3, 2014

I shall be submitting a PR to get this started

@timowest timowest added the feature label Aug 4, 2014

@timowest timowest added this to the 3.4.3 milestone Aug 14, 2014

@timowest timowest closed this Aug 14, 2014

@timowest

This comment has been minimized.

Show comment
Hide comment
@timowest

timowest Aug 31, 2014

Member

Released in 3.4.3

Member

timowest commented Aug 31, 2014

Released in 3.4.3

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