Re: Partitioned tables and relfilenode
Kyotaro HORIGUCHI <horiguchi.kyotaro@lab.ntt.co.jp>
From: Kyotaro HORIGUCHI <horiguchi.kyotaro@lab.ntt.co.jp>
To: Langote_Amit_f8@lab.ntt.co.jp
Cc: amitlangote09@gmail.com, robertmhaas@gmail.com, m.milyutin@postgrespro.ru,
pgsql-hackers@postgresql.org
Date: 2017-03-31T08:33:26Z
Lists: pgsql-hackers
At Fri, 31 Mar 2017 16:17:05 +0900, Amit Langote <Langote_Amit_f8@lab.ntt.co.jp> wrote in <2dec1acb-6e2f-5aa5-0e26-fcc172ce9359@lab.ntt.co.jp> > Horiguchi-san, > > On 2017/03/31 15:50, Kyotaro HORIGUCHI wrote: > > At Thu, 30 Mar 2017 20:58:35 +0900, Amit Langote wrote: > >> Updated patch attached. > > > > Thank you. > > > > - Applies cleanly on master (f90d23d) > > - Compiled without error > > - Code seems fine. > > - Documentaion seems fine.. for me. > > - Passes regression test. > > - Leaving the ALTER-on-toast.* problem is fine for me. > > Thanks for the review. > > > The regression contains the tests to fail with several reloptions > > only for partitioned tables. Are they still required even though > > it is now in the same framework with other kind of reloptions? > > Hmm, maybe we don't really need those tests, so removed. I have no more comment on this. Thank you. regards, -- Kyotaro Horiguchi NTT Open Source Software Center
Commits
-
Code review for c94e6942cefe7d20c5feed856e27f672734b1e2b.
- 1d5fede4a900 10.0 landed
-
Don't allocate storage for partitioned tables.
- c94e6942cefe 10.0 landed
-
Don't scan partitioned tables.
- d3cc37f1d801 10.0 landed
-
Don't uselessly rewrite, truncate, VACUUM, or ANALYZE partitioned tables.
- 3c3bb99330aa 10.0 landed