copy command bug - Mailing list pgsql-bugs

From Mary LaClair
Subject copy command bug
Date
Msg-id e831d360-c991-3078-01e6-47533df4c179@caliper.com
Whole thread Raw
Responses Re: copy command bug  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-bugs
The copy command fails to load a large csv file (>2GB) with a "Could not stat file" error.  This occurs on Windows with all versions since 10.7.
It happens because fstat() on Windows fails with large csv files. 

Example:
COPY table_name FROM 'filename.csv' WITH CSV HEADER
succeeds with small files and fails with larger ones.

In my tests, using other methods to load the file, e.g.
COPY table_name FROM PROGRAM 'cmd /c \"type filename.csv"' WITH CSV HEADER
succeeds with all file sizes but runs 10-30% slower, which is a significant difference.

Thanks, Mary

Mary LaClair

Vice President, Software Development | Caliper Corporation

|||||||||||||||||||||||||||||||||||||||||||||||||

1172 Beacon St, Ste 300 • Newton MA 02461 USA

Direct: 617-340-2003 • Main: 617-527-4700

mary@caliper.comwww.caliper.com
P.S.
I downloaded the  master source and implemented a patch but was unable to test because I do not have a version of Visual Studio (11 or 12) old enough to try it:
Here's the change to BeginCopyFrom() in copy.c, starting at line 3529:
  {
            unsigned short st_mode;

            cstate->copy_file = AllocateFile(cstate->filename, PG_BINARY_R);
            if (cstate->copy_file == NULL)
            {
                /* copy errno because ereport subfunctions might change it */
                int            save_errno = errno;

                ereport(ERROR,
                        (errcode_for_file_access(),
                         errmsg("could not open file \"%s\" for reading: %m",
                                cstate->filename),
                         (save_errno == ENOENT || save_errno == EACCES) ?
                         errhint("COPY FROM instructs the PostgreSQL server process to read a file. "
                                 "You may want a client-side facility such as psql's \\copy.") : 0));
            }

#ifndef WIN32
            {
            struct _stat32i64 st;
            // need the i64 version to handle files > 1GB
            if (_fstat32i64(fileno(cstate->copy_file), &st))
                ereport(ERROR,
                        (errcode_for_file_access(),
                         errmsg("could not stat file \"%s\": %m",
                                cstate->filename)));
            st_mode = st.st_mode;
            }
#else
            {
            struct stat st;
   
        if (fstat(fileno(cstate->copy_file), &st))
                ereport(ERROR,
                        (errcode_for_file_access(),
                         errmsg("could not stat file \"%s\": %m",
                                cstate->filename)));
            st_mode = st.st_mode;
            }
#endif

            if (S_ISDIR(st_mode))
                ereport(ERROR,
                        (errcode(ERRCODE_WRONG_OBJECT_TYPE),
                         errmsg("\"%s\" is a directory", cstate->filename)));
        }


pgsql-bugs by date:

Previous
From: PG Bug reporting form
Date:
Subject: BUG #16676: SELECT ... FETCH FIRST ROW WITH TIES FOR UPDATE SKIP LOCKED locks rows it doesn't return
Next
From: Tom Lane
Date:
Subject: Re: copy command bug