Re: updated SORT/LIMIT patch

Gregory Stark wrote:

> Attached is a small patch which fixes this case. It also makes the check
> slightly more liberal -- we don't need to resort if the previous sort was
> unbounded or the bound was greater than or equal to the new bound.

Huh, can you clarify this comment:

+        * XXX It would be nice to check tuplesortstate->boundUsed too but that
+        * seems like an abstraction violation. And for that matter to check
+        * the tuplesort to see if randomaccess is possible even if it wasn't
+        * requested so we don't resort input when the parameters haven't
+        * changed if it was sorted in memory.

I'm having serious trouble parsing it.

Thanks.

--
Alvaro Herrera                                http://www.CommandPrompt.com/
PostgreSQL Replication, Consulting, Custom Development, 24x7 support