Document the new JDBC legacyPreparedStatements flag#19502
Document the new JDBC legacyPreparedStatements flag#19502mosabua merged 2 commits intotrinodb:masterfrom
Conversation
mosabua
left a comment
mosabua
left a comment
There was a problem hiding this comment.
Also add "Defaults to ??" in the description.
docs/src/main/sphinx/client/jdbc.md
Outdated
There was a problem hiding this comment.
Not sure we want this code level info .. maybe .. given its the JDBC driver. What do you think @electrum ?
There was a problem hiding this comment.
I would still like to get an opinion from @electrum to see if this kind of Java code level doc is desired for the JDBC driver docs. I can see arguments for yes and no..
Maybe its best to make sure this kind of stuff is in the javadocs at least so developers find out about it where they work? Wdyt about that idea @lpoulain ?
lpoulain
force-pushed
the
document-jdbc-legacy-prepared-statements
branch
from
fc01508 to
b56a5a7
Compare
Co-authored-by: Manfred Moser <manfred@simpligility.ca>
|
I am merging this so that we have docs for the already merged code. If we end up renaming we just need to update code and docs. |
mosabua
left a comment
mosabua
left a comment
There was a problem hiding this comment.
Looks good for now. If we rename and/or want Java method info we can do a separate follow up.
3 participants
Description
Docs for #19022
Additional context and related issues
Release notes
( ) This is not user-visible or is docs only, and no release notes are required.
( ) Release notes are required. Please propose a release note for me.
( ) Release notes are required, with the following suggested text: