Re: [HACKERS] pg_dump insert with column names speedup

2013-11-15 Thread Tom Lane
David Rowley writes: > The tailing white space is fixed in the attached patch. Applied with minor cosmetic adjustments. regards, tom lane -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.o

Re: [HACKERS] pg_dump insert with column names speedup

2013-11-15 Thread David Rowley
On Sat, Nov 16, 2013 at 9:04 AM, Peter Eisentraut wrote: > src/bin/pg_dump/pg_dump.c:1604: trailing whitespace. > + staticStmt = createPQExpBuffer(); > src/bin/pg_dump/pg_dump.c:1612: trailing whitespace. > + else > src/bin/pg_dump/pg_du

Re: [HACKERS] pg_dump insert with column names speedup

2013-11-15 Thread Peter Eisentraut
src/bin/pg_dump/pg_dump.c:1604: trailing whitespace. + staticStmt = createPQExpBuffer(); src/bin/pg_dump/pg_dump.c:1612: trailing whitespace. + else src/bin/pg_dump/pg_dump.c:1614: trailing whitespace. +

Re: [HACKERS] pg_dump insert with column names speedup

2013-10-04 Thread David Rowley
On Sat, Oct 5, 2013 at 6:39 AM, Stephen Frost wrote: > * Robert Haas (robertmh...@gmail.com) wrote: > > On Fri, Oct 4, 2013 at 11:28 AM, Tom Lane wrote: > > > David Rowley writes: > > >> Here's a small patch which greatly increases the speed of > > >> pg_dump --column-inserts. > > > > > > The r

Re: [HACKERS] pg_dump insert with column names speedup

2013-10-04 Thread Stephen Frost
* Robert Haas (robertmh...@gmail.com) wrote: > On Fri, Oct 4, 2013 at 11:28 AM, Tom Lane wrote: > > David Rowley writes: > >> Here's a small patch which greatly increases the speed of > >> pg_dump --column-inserts. > > > > The reason why no one's paid any attention to the speed of that code path

Re: [HACKERS] pg_dump insert with column names speedup

2013-10-04 Thread Robert Haas
On Fri, Oct 4, 2013 at 11:28 AM, Tom Lane wrote: > David Rowley writes: >> Here's a small patch which greatly increases the speed of >> pg_dump --column-inserts. > > The reason why no one's paid any attention to the speed of that code path > is that if you care about dump/restore speed, you shoul

Re: [HACKERS] pg_dump insert with column names speedup

2013-10-04 Thread Tom Lane
David Rowley writes: > Here's a small patch which greatly increases the speed of > pg_dump --column-inserts. The reason why no one's paid any attention to the speed of that code path is that if you care about dump/restore speed, you should be using the COPY code paths instead. Is it really worth

[HACKERS] pg_dump insert with column names speedup

2013-10-04 Thread David Rowley
Here's a small patch which greatly increases the speed of pg_dump --column-inserts. Previous to this patch a string was being build for the names of the columns for each row in the table... I've now changed this to only do this once for the table. I also did some clean ups to remove calling the v