Re: foreign partition DDL regression tests
Amit Langote <langote_amit_f8@lab.ntt.co.jp>
From: Amit Langote <Langote_Amit_f8@lab.ntt.co.jp>
To: Robert Haas <robertmhaas@gmail.com>
Cc: Pg Hackers <pgsql-hackers@postgresql.org>
Date: 2017-03-09T00:52:53Z
Lists: pgsql-hackers
Attachments
On 2017/03/09 1:28, Robert Haas wrote: > On Tue, Feb 21, 2017 at 8:40 PM, Amit Langote > <Langote_Amit_f8@lab.ntt.co.jp> wrote: >> Ashutosh Bapat pointed out [0] that regression tests are missing for the >> foreign partition DDL commands. Attached patch takes care of that. > > Committed. Thanks. > I didn't do anything about Ashutosh's comment that we could use ALTER > FOREIGN TABLE rather than ALTER TABLE someplace; that didn't seem > critical. Attached is a patch to fix that, just in case. > Also, the names of the objects in this test are kinda generic (pt2 et. > al.) but they match the existing names in the same file (pt1, foo). > If we're going to start differentiating those a little better, we > should probably change them all, and as a separate commit. Agreed. Thanks, Amit
Commits
-
Document lack of validation when attaching foreign partitions.
- b54aad8e34bd 10.0 landed
-
Add tests for foreign partitions.
- 0d130c7abc85 10.0 landed