Tom Lane wrote: > Alvaro Herrera <alvhe...@2ndquadrant.com> writes: > > Tom Lane wrote: > >> I'm inclined to let it call heap_setscanlimits only if not allow_sync. > > > It is possible for a partial range scan to join an existing herd of > > scans that happens to be processing that part of the table, in which > > case this wouldn't be sufficient. However, two considerations: 1) range > > scans, at least for BRIN, aren't normally large enough for synscans to > > matter all that much; and 2) it would require additional code. So I'm > > inclined to do it as you suggest, which is simplest. > > I already did that, but feel free to whack it further if you're so > inclined.
Nah, I just failed to see your commit. -- Álvaro Herrera http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers