Skip to content

Commit

Permalink
update notes.
Browse files Browse the repository at this point in the history
--HG--
extra : convert_revision : 73ba6bae11bff617699e5022b5b430cf5c6bb5a0
  • Loading branch information
waterson committed Apr 18, 2010
1 parent 12363c1 commit 20b0074
Showing 1 changed file with 3 additions and 10 deletions.
13 changes: 3 additions & 10 deletions NOTES
Expand Up @@ -37,6 +37,9 @@ Caveats
interpret the times using the current connection's default
time zone setting, instead. Is that better? Or worse?

* Regardless, the types I convert to are now deprecated, and I need
to implement the new types (SqlLocalDate, etc.)

* Out of the box, MySQL probably uses MyISAM tables to store its
data, and MyISAM tables don't support transactions. Yet, I'm
going to blindly respond "yes" if you ask whether the driver
Expand All @@ -50,12 +53,6 @@ Caveats
and a MySQL BLOB column. If you ask about the metadata of either,
I'll tell you it's a SqlBinaryT.

* I have not tested this with non-ASCII data. I don't pay any
attention to the connection character set. I use withCString,
which doesn't pay any attention, either. It may be that this
total lack of attention results in things serendipitously working
if the MySQL connection defaults to using UTF-8 as the encoding.

* The statement and table metadata could stand to be improved a bit.
In particular, it would be nice if "describeTable foo" and
"describeResults" on "SELECT * FROM foo" returned the same thing.
Expand All @@ -71,10 +68,6 @@ Caveats
statement, explicitly disconnects the connection, and then tries
to play with the statement.

* I need to verify that embedded null characters on bound parameter
strings work correctly. I'm using "withCString", when I probably
should be using "withCStringLen".

* It probably makes sense to marshall to the SqlByteString type when
retrieving BLOB data.

Expand Down

0 comments on commit 20b0074

Please sign in to comment.