True, but you have to forcibly call closeCursor. This code assumes that the
desired result is to only cache at the end of the cursor but it could be
that the programmer wants the cursor closed early and expects that to be
cached.

I obviously could be completely wrong,  just throwing thoughts around!
On 2 Apr 2013 17:51, "Christophe Coevoet" <notifications@github.com> wrote:

> In lib/Doctrine/DBAL/Cache/ResultCacheStatement.php:
>
> > @@ -217,6 +218,7 @@ public function fetchColumn($columnIndex = 0)
> >              // TODO: verify this is correct behavior
> >              return false;
> >          }
> > +        $this->atEndOfStatement = true;
>
> @Alan01252 <https://github.com/Alan01252> If you save in cache before
> reaching the end of the cursor, you will have incomplete data when using
> the cache the next time
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/doctrine/dbal/pull/288/files#r3621776>
> .
>