Bryan Pendleton <[email protected]> writes:

> Does this mean that we should be calling this.close() rather than 
> source.close()?

Or possibly in addition to source.close(). PRRS.close() usually calls
both source.close() and super.close().

I've filed DERBY-6268 to track this bug.

> thanks,
>
> bryan
>
> On Mon, Jun 17, 2013 at 8:51 AM, Knut Anders Hatlen <[email protected]> 
> wrote:
>
>     shortcut so that NoPutResultSetImpl.close() is not invoked:
>    
>         /* Nothing to do if open was short circuited by false constant 
> expression */
>         if (shortCircuitOpen)
>         {
>                 isOpen = false;
>                 shortCircuitOpen = false;
>                 source.close();
>                 return;
>         }
>    
>     Notice that it doesn't call super.close() in this case. I suspect that
>     this is what's causing the issue you're seeing.
>

-- 
Knut Anders

Reply via email to