| From: | Mikko Tiihonen <mikko(dot)tiihonen(at)nitorcreations(dot)com> |
|---|---|
| To: | pgsql-jdbc(at)postgresql(dot)org |
| Subject: | Binary transfer patches v15 |
| Date: | 2011-09-21 22:22:50 |
| Message-ID: | 4E7A63BA.1020505@nitorcreations.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-jdbc |
I believe these are now ready for merging to cvs head. The patch set now passes all tests on jdk7u2 + postgres 9.1.0.
Changes from last week:
- Rebased to latest cvs
- Fixed connection leak in jdbc3/DatabaseMetaDataTest.java
=> now it is possible to run the unit tests with max_connections=2 in postgresql.conf
- Fixed ForceBinaryTransfer bug in DatabaseEncodingTest
=> Required a new field to Jdbc2ResultSet to keep the real statement alive
(Thanks to Oliver for the good analysis on the problem)
- Fixed UpdateableResultTest.testUpdateStreams with binary transfers
=> now ResultSet.refreshRow() informs lower level code that the results are for
updateable result set and should not use binary transfer
Any comments on what still needs to be done before this is accepted?
-Mikko
| Attachment | Content-Type | Size |
|---|---|---|
| 1-bugfixes.patch | text/plain | 1.4 KB |
| 2-binarytransfer-core.patch | text/plain | 35.1 KB |
| 3-debugging.patch | text/plain | 6.0 KB |
| 4-integertypes.patch | text/plain | 13.9 KB |
| 5-datetypes.patch | text/plain | 23.3 KB |
| 6-more-tests.patch | text/plain | 1.6 KB |
| 7-ide-testing.patch | text/plain | 2.2 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Dave Cramer | 2011-09-21 22:44:24 | Re: Binary transfer patches v15 |
| Previous Message | Radosław Smogura | 2011-09-21 14:00:03 | Re: behavior at the end of a transaction |