Log In - Home Page

Patch: SQLDA support for ECPG

Edit Patch - Move To Another CommitFest - Delete Patch

CommitFest 2010-01
Topic ECPG
Patch Status Committed
Author Zoltan Boszormenyi
Reviewers Michael Meskes
Committer Michael Meskes
Close Date 2010-01-05
Comments
Patch by zboszor on 2009-09-03 08:04:30 AM: Initial version.
Patch by zboszor on 2009-09-18 01:31:49 AM: Changes:
- removed a decimal-related leak fix (to be posted separately)
- free(sqlda->sqlvar) as well if it's not in the same allocation area
as the main sqlda
Comment by nmisch on 2009-09-30 04:17:58 PM: Will review on 2009-10-03.
Review by nmisch on 2009-10-04 12:21:20 AM: Minor patch rejects, a test suite failure. Not a comprehensive review.
Patch by zboszor on 2009-10-05 07:38:29 AM: New patch to fix rejects.
The test suite needs "make install" before it can succeed.
Patch by zboszor on 2009-10-08 04:53:47 AM: Set sqllen, sqlind, sqlilen and sqlitype per-field properties
SQLINT8 and SQLSERIAL8 are ECPGt_long on 64-bit, not ECPGt_long_long
Comment by nmisch on 2009-10-10 11:44:35 AM: Wrote to author; I will not complete a review by 2009-10-10. Plan to do so when time permits.
Patch by zboszor on 2009-11-16 05:24:57 AM: Rebased to current CVS
Patch by zboszor on 2009-12-14 07:31:15 AM: - SQLDA is now an ECPG native feature, not compat-only.
- As a consequence, and to be coherent with the Informix syntax,
keyword "SQL" in SQL DESCRIPTOR is now mandatory
to denote the named SQL descriptor. "DESCRIPTOR"
without "SQL" means SQLDA descriptor
- SQLDA can now return more than one tuples using
the ->desc_next pointer
- Bcause SQLDA is now a native feature, there are two
sqlda.pgc regression tests, to test compat-only syntax in
compat mode and multi-tuple return values in native mode.
- sqlda->sqlvar[i].sqltype uses ECPGt_* type symbols,
no more funny-looking typedefs in sqltypes.h, it's a clean
compat-only header, as it was before
Patch by zboszor on 2009-12-15 03:59:31 AM: New version, small fixes.
Patch by zboszor on 2009-12-16 08:59:25 AM: Rebased to current CVS after the auto-prepare fix was applied.
Patch by zboszor on 2010-01-05 02:22:53 AM: DB2/Sybase compatible SQLDA was added for native mode
Informix-specific FETCH ... USING DESCRIPTOR sqlda is accepted for native mode

Add Comment

Please log in to comment on this patch.