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