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

Manual should stop recommending use of deprecated DefaultExecuteListener or DefaultRecordListener #15511

Closed
cies opened this issue Aug 28, 2023 · 4 comments

Comments

@cies
Copy link
Contributor

cies commented Aug 28, 2023

The documentation URL

https://www.jooq.org/doc/latest/manual/sql-execution/execute-listeners/

Feedback

On that page I read:

For convenience and better backwards-compatibility, consider extending org.jooq.impl.DefaultExecuteListener instead of implementing this interface.

And:

// Extending DefaultExecuteListener, which provides empty implementations for all methods...

Once I followed the advice I found that DefaultExecuteListener is deprecated.

Maybe there's more cases in the docs.

@lukaseder
Copy link
Member

Thanks for your hint. Will look into this soon.

@lukaseder lukaseder changed the title DefaultExecutionListener is deprecated since 3.17 but is still advertized as the way to go Manual should stop recommending use of deprecated DefaultExecuteListener Aug 28, 2023
@lukaseder lukaseder changed the title Manual should stop recommending use of deprecated DefaultExecuteListener Manual should stop recommending use of deprecated DefaultExecuteListener or DefaultRecordListener Aug 28, 2023
@lukaseder
Copy link
Member

I believe I got them all now. Thanks again for your report!

3.19 Other improvements automation moved this from To do to Done Aug 28, 2023
@cies
Copy link
Contributor Author

cies commented Aug 28, 2023

I'm glad I can help. Thank you for jooq!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Development

No branches or pull requests

2 participants