On 2012-11-21 12:54:30 -0500, Robert Haas wrote: > On Wed, Nov 21, 2012 at 9:46 AM, Robert Haas <[email protected]> wrote: > > I guess I'll take another whack at it. > > New version attached.
I think the assert in replace_first should be Assert(!binaryheap_empty(heap) && heap->has_heap_property); instead of Assert(heap->has_heap_property); looks good otherwise. Thanks, Andres -- Andres Freund http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training & Services -- Sent via pgsql-hackers mailing list ([email protected]) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
