From: | Zhihong Yu <zyu(at)yugabyte(dot)com> |
---|---|
To: | Israel Barth Rubio <barthisrael(at)gmail(dot)com> |
Cc: | Andrew Dunstan <andrew(at)dunslane(dot)net>, pgsql-hackers(at)lists(dot)postgresql(dot)org |
Subject: | Re: Add support for DEFAULT specification in COPY FROM |
Date: | 2022-09-26 15:23:02 |
Message-ID: | CALNJ-vQeoRAVzi4cjpPHc0xesZnhrbTVz6PWnNJuBrFFk6eFBA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Mon, Sep 26, 2022 at 8:12 AM Israel Barth Rubio <barthisrael(at)gmail(dot)com>
wrote:
> Hello Andrew,
>
> > . There needs to be a check that this is being used with COPY FROM, and
> > the restriction needs to be stated in the docs and tested for. c.f.
> > FORCE NULL.
> >
> > . There needs to be support for this in psql's tab_complete.c, and
> > appropriate tests added
> >
> > . There needs to be support for it in contrib/file_fdw/file_fdw.c, and a
> > test added
> >
> > . The tests should include psql's \copy as well as sql COPY
> >
> > . I'm not sure we need a separate regression test file for this.
> > Probably these tests can go at the end of src/test/regress/sql/copy2.sql.
>
> Thanks for your review! I have applied the suggested changes, and I'm
> submitting the new patch version.
>
> Kind regards,
> Israel.
>
Hi,
+ /* attribute is NOT to be copied from input */
I think saying `is NOT copied from input` should suffice.
+ defaults = (bool *) palloc0(num_phys_attrs * sizeof(bool));
+ MemSet(defaults, false, num_phys_attrs * sizeof(bool));
Is the MemSet() call necessary ?
+ /* fieldno is 0-index and attnum is 1-index */
0-index -> 0-indexed
Cheers
From | Date | Subject | |
---|---|---|---|
Next Message | Robert Haas | 2022-09-26 15:27:52 | Re: has_privs_of_role vs. is_member_of_role, redux |
Previous Message | James Coleman | 2022-09-26 15:17:01 | Re: Consider parallel for lateral subqueries with limit |