Peter Geoghegan <p...@heroku.com> writes: > While working on my parallel CREATE INDEX patch, I came across a > problem.
I took a quick look at this. I do not follow the logic in this new bit: + if (availMemLessRefund <= + (int64) state->activeTapes * ALLOCSET_DEFAULT_INITSIZE) + return; It doesn't seem to me that this limit has anything to do with anything, and the comment claiming only that it's "noncritical" isn't helping. If the point is to prevent the later LACKMEM check from failing, then certainly there is something critical somewhere. I'd rather see this explained as "we need at least X, but we choose to require at least Y to avoid repalloc thrashing". And maybe the code should use Max(X,Y) rather than blindly assuming that ALLOCSET_DEFAULT_INITSIZE exceeds whatever the true minimum is. regards, tom lane -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers