On Mon, Dec 19, 2016 at 10:59 PM, Robert Haas <robertmh...@gmail.com> wrote: > On Sun, Dec 18, 2016 at 10:00 PM, Amit Langote > <langote_amit...@lab.ntt.co.jp> wrote: >> Here are updated patches including the additional information. > > Thanks. Committed 0001. Will have to review the others when I'm less tired.
0002. Can you add a test case for the bug fixed by this patch? 0003. Loses equalTupleDescs() check and various cases where ExecOpenIndexes can be skipped. Isn't that bad? Also, "We locked all the partitions above including the leaf partitions" should say "Caller must have locked all the partitions including the leaf partitions". 0004. Unnecessary whitespace change in executor.h. Still don't understand why we need to hoist RelationGetPartitionQual() into the caller. 0005. Can you add a test case for the bug fixed by this patch? -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
