On Wed, Mar 30, 2016 at 7:51 AM, Dilip Kumar <dilipbal...@gmail.com> wrote:
> + if (lockWaiters) > + /* > + * Here we are using same freespace for all the Blocks, but that > + * is Ok, because all are newly added blocks and have same freespace > + * And even some block which we just added to FreespaceMap above, is > + * used by some backend and now freespace is not same, will not harm > + * anything, because actual freespace will be calculated by user > + * after getting the page. > + */ > + UpdateFreeSpaceMap(relation, firstBlock, blockNum, freespace); > > > Does this look good ? Done in better way.. + lockWaiters = RelationExtensionLockWaiterCount(relation); + + if (lockWaiters == 0) + return; + -- Regards, Dilip Kumar EnterpriseDB: http://www.enterprisedb.com
multi_extend_v21.patch
Description: Binary data
-- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
