[freetds] Problem retrieveing results from bottom to top
Daniel A. Veiga
dveiga at advtechnology.com.ar
Thu Jun 19 18:11:09 EDT 2008
Sory for the delay in answering, I had to travel and I didn´t
have access to my computer. I'm attaching the patch in the
format you asked for.
I don't think the patch is going to create problems, because
as you can see from the code, if something goes wrong,
SQLGetStmtAttr(SQL_ATTR_ROW_NUMBER) returns 0 (unknown) that's
what it always returned before this change.
Please let me know if you need anything else. I haven't used
diff too much, so if its still not what you're looking for, I
can always generate a new one. Bye
> On Jun 17, 6:28pm, jklowden at freetds.org ("James K. Lowden") wrote:
> -- Subject: Re: [freetds] Problem retrieveing results from bottom to
> | Daniel A. Veiga wrote:
> | > I believe the approach of MSSQL ODBC driver (fetching the row
> | > number only when needed) is better than the one I found in the
> | > jDBC driver. So I implemented
> | > SQLGetStmtAttr(SQL_ATTR_ROW_NUMBER) that way.
> | Hi Daniel,
> | I don't want your patch to get lost; it's nice work. When you don't
> | an answer to serious work on this list for a few days, it's usually
> | because it need serious consideration and no one's been able to look
> at it
> | carefully.
> | Your patch is a little difficult to apply easily because of how it's
> | generated. Could I ask you to redo it?
> | The command you want is:
> | $ diff -u /path/to/freetds-orig . > odbc.cursor7.diff
> | while '.' represents the top of your patched FreeTDS source tree.
> | A universal diff is much easier to review (for these eyes, at least)
> | easier to edit and apply. If you don't do that, I'll have to apply
> | patch one at at time, then use CVS to generate a universal (-u) diff,
> | start again.
> | Unless Frediano objects, I expect to review and apply your patch more
> | less as is and let the chips fall where they may. If it creates a
> | problem, we can always back it out. Of course, I'm not expecting
> | problems.
> diff -ru
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
More information about the FreeTDS