Pass modern COPY syntax to backend, since copy (query) does not accept old syntax...
authorAndrew Dunstan <andrew@dunslane.net>
Mon, 5 Feb 2007 15:22:18 +0000 (15:22 +0000)
committerAndrew Dunstan <andrew@dunslane.net>
Mon, 5 Feb 2007 15:22:18 +0000 (15:22 +0000)
src/bin/psql/copy.c

index 879a20cac7fa4e5993ee4a00a3d0b1c12dd545ed..e655f46c349a3d00a89809a565150d70e9a1926f 100644 (file)
@@ -3,7 +3,7 @@
  *
  * Copyright (c) 2000-2007, PostgreSQL Global Development Group
  *
- * $PostgreSQL: pgsql/src/bin/psql/copy.c,v 1.72 2007/01/05 22:19:49 momjian Exp $
+ * $PostgreSQL: pgsql/src/bin/psql/copy.c,v 1.73 2007/02/05 15:22:18 adunstan Exp $
  */
 #include "postgres_fe.h"
 #include "copy.h"
@@ -268,7 +268,7 @@ parse_slash_copy(const char *args)
                    0, false, false, pset.encoding);
 
    /*
-    * Allows old COPY syntax for backward compatibility 2002-06-19
+    * Allows old COPY syntax for backward compatibility.
     */
    if (token && pg_strcasecmp(token, "using") == 0)
    {
@@ -480,32 +480,28 @@ do_copy(const char *args)
 
    printfPQExpBuffer(&query, "COPY ");
 
-   /* Uses old COPY syntax for backward compatibility 2002-06-19 */
-   if (options->binary)
-       appendPQExpBuffer(&query, "BINARY ");
-
    appendPQExpBuffer(&query, "%s ", options->table);
 
    if (options->column_list)
        appendPQExpBuffer(&query, "%s ", options->column_list);
 
-   /* Uses old COPY syntax for backward compatibility 2002-06-19 */
-   if (options->oids)
-       appendPQExpBuffer(&query, "WITH OIDS ");
-
    if (options->from)
        appendPQExpBuffer(&query, "FROM STDIN");
    else
        appendPQExpBuffer(&query, "TO STDOUT");
 
 
-   /* Uses old COPY syntax for backward compatibility 2002-06-19 */
+   if (options->binary)
+       appendPQExpBuffer(&query, " BINARY ");
+
+   if (options->oids)
+       appendPQExpBuffer(&query, " OIDS ");
+
    if (options->delim)
-       emit_copy_option(&query, " USING DELIMITERS ", options->delim);
+       emit_copy_option(&query, " DELIMITER ", options->delim);
 
-   /* There is no backward-compatible CSV syntax */
    if (options->null)
-       emit_copy_option(&query, " WITH NULL AS ", options->null);
+       emit_copy_option(&query, " NULL AS ", options->null);
 
    if (options->csv_mode)
        appendPQExpBuffer(&query, " CSV");