-
Notifications
You must be signed in to change notification settings - Fork 95
Document how to replace the SqlClientPool using services #157
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
Comments
Yes, I agree, but perhaps we need to be a little bit careful about that because I'm not sure that the So I was thinking of leaving it undocumented for now, and just having it there so that people who ask for it explicitly can be pointed to it with a big caveat that they should not rely on it not breaking. |
Sounds good. It's not something we need immediately. ORM uses some annotation to highlight features that are quite experimental and might change in the future. We can introduce something similar. I will just keep the issue as a reminder for when we release the first final. |
That sounds like a good idea. But why can't we just use the same annotations? We already have a hard dependency on Hibernate ORM. |
Yes, of course :) |
@DavideD would you consider it sufficient documentation that this is described in the Javadoc for |
ok |
soooo .... does that mean we can close this? :-) |
Now that we have #155 we need to remember to add an explanation with example on how to do that in the documentation (When we start to work on one)
The text was updated successfully, but these errors were encountered: