file-fdw and force_null
Hello Friends
I tried to create a file-fdw foreign table with force_null for some columns.
FORCE_NULL syntax differs from COPY. For me it was not clear that the
"OPTIONS" keyword by the column was necessary, so it would be nice to add
this to the example.
DROP FOREIGN TABLE IF EXISTS testforcenull;
CREATE FOREIGN TABLE testforcenull
( cola text NOT NULL,
colb text,
colc text OPTIONS(FORCE_NULL 'true')
) SERVER bevcsv
OPTIONS (filename '~/testforcenull.csv',
FORMAT 'csv',
HEADER 'true',
DELIMITER ';',
QUOTE '"'
);
best regards
On 11 Sep 2023, at 16:28, Boshomi Phenix <boshomi@gmail.com> wrote:
I tried to create a file-fdw foreign table with force_null for some columns.
FORCE_NULL syntax differs from COPY. For me it was not clear that the "OPTIONS" keyword by the column was necessary, so it would be nice to add this to the example.
Considering that we don't really have the OPTIONS syntax documented in an
example, and that we reference COPY heavily on this page, I tend to agree.
How about the attached?
Looking at this it's a bit odd that we use <example> here which is very
infrequently used. It might be worth replacing this to make the docs more
uniform, but that's for another patch (and thread), for now it's using the same
syntax already present here.
--
Daniel Gustafsson
Attachments:
file_fdw_options.diffapplication/octet-stream; name=file_fdw_options.diff; x-unix-mode=0644Download+16-0
Am Di., 12. Sept. 2023 um 12:25 Uhr schrieb Daniel Gustafsson <
daniel@yesql.se>:
Considering that we don't really have the OPTIONS syntax documented in an
example, and that we reference COPY heavily on this page, I tend to agree.
How about the attached?
Yes, this looks good to me.
Thank you!
On 13 Sep 2023, at 22:10, Boshomi Phenix <boshomi@gmail.com> wrote:
Am Di., 12. Sept. 2023 um 12:25 Uhr schrieb Daniel Gustafsson <daniel@yesql.se <mailto:daniel@yesql.se>>:
Considering that we don't really have the OPTIONS syntax documented in an
example, and that we reference COPY heavily on this page, I tend to agree.
How about the attached?Yes, this looks good to me.
I've applied this today, thanks for the report.
--
Daniel Gustafsson