From: | Josh Berkus <josh(at)agliodbs(dot)com> |
---|---|
To: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
Cc: | Itagaki Takahiro <itagaki(dot)takahiro(at)oss(dot)ntt(dot)co(dot)jp>, Jaime Casanova <jcasanov(at)systemguards(dot)com(dot)ec>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: COPY WITH CSV FORCE QUOTE * -- REVIEW |
Date: | 2009-07-16 18:47:15 |
Message-ID: | 4A5F75B3.9020405@agliodbs.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
All,
1) Patch applies cleanly against CVS head.
2) Patch compiles and builds cleanly.
3) Unable to check docs because of general doc build problems.
4) Tested the following commands, using a 10MB table of PostgreSQL log data:
postgres=# COPY marchlog TO '/tmp/marchlog1.csv' with csv header;
COPY 81097
postgres=# COPY marchlog TO '/tmp/marchlog2.csv' with csv header force
quote *;
COPY 81097
postgres=# COPY marchlog TO '/tmp/marchlog3.csv' with csv header force
quote process_id;
COPY 81097
postgres=# COPY marchlog TO '/tmp/marchlog4.csv' with csv force quote *;
COPY 81097
postgres=# COPY marchlog TO '/tmp/marchlog5.csv' with force quote *;
ERROR: COPY force quote available only in CSV mode
STATEMENT: COPY marchlog TO '/tmp/marchlog5.csv' with force quote *;
ERROR: COPY force quote available only in CSV mode
postgres=# COPY reloadlog FROM '/tmp/marchlog2.csv' with csv header;
COPY 81097
postgres-# \copy marchlog TO '/tmp/marchlog5.csv' with csv force quote *;
postgres-#
Per discussion, I did not test FORCE QUOTE NOT NULL *.
All output looked as expected. This patch did not seem to change
eariler functionality, and seems to quote as specified.
Unless there are other things we want to test (CLOBs?) I think the patch
is probably ready for code review of the FORCE QUOTE * portion.
--
Josh Berkus
PostgreSQL Experts Inc.
www.pgexperts.com
From | Date | Subject | |
---|---|---|---|
Next Message | Bruce Momjian | 2009-07-16 18:58:20 | Re: [GENERAL] pg_migrator not setting values of sequences? |
Previous Message | Rick Gigger | 2009-07-16 18:45:26 | Re: Synch Rep for CommitFest 2009-07 |