Re: patch: fix ResultSet.isLast and friends with cursor-based

On Sat, 10 Apr 2004, Oliver Jowett wrote:

> The attached patch fixes (and adds testcases for) the various resultset
> position querying methods (isFirst, isLast, isBeforeFirst, isAfterLast)
> that can be called on any resultset. The current driver does not handle
> these methods correctly when a TYPE_FORWARD_ONLY resultset that is
> backed by a cursor is used.
>
> isLast() may be more expensive after this patch (as it may need to do a
> FETCH), but this cost is warned about in the JDBC javadoc.
>
> A couple of other changes also in this patch:
>
> - prevent use of relative row movement methods (relative, next,
> previous) when on the insert row -- as I read the spec these shouldn't
> be allowed.
> - reset onInsertRow whenever an absolute row positioning method is called.
>

Patch Applied.

Kris Jurka