Re: [HACKERS] [ patch ] pg_dump: new --custom-fetch-table and--custom-fetch-value parameters - Mailing list pgsql-hackers

From Stephen Frost
Subject Re: [HACKERS] [ patch ] pg_dump: new --custom-fetch-table and--custom-fetch-value parameters
Date
Msg-id 20161221174448.GJ18360@tamriel.snowman.net
Whole thread Raw
In response to [HACKERS] [ patch ] pg_dump: new --custom-fetch-table and--custom-fetch-value parameters  ("Andrea Urbani" <matfanjol@mail.com>)
List pgsql-hackers
Andrea,

* Andrea Urbani (matfanjol@mail.com) wrote:
>    I had a problem with a Postgresql 9.3.5 on 32 bit linux, old 2.6.26
>    kernel:

Ok, though, to be clear, this is a feature request, so we wouldn't
back-patch adding this to pg_dump.

>    I have solve it adding two new parameters, --custom-fetch-table and
>    --custom-fetch-value, to fetch less records for the specified table(s).

Giving the user the ability to change the fetch size sounds interesting,
though do we really need to specify it per table?  What about just a
--fetch-size=X option?

>    This does not completely solve the problem, but it helps you to get more
>    chance to be able to dump your database.

That is certainly a worthwhile goal.

>        pg_dump --dbname=healthorganizer --username=hor --column-inserts
>    --custom-fetch-table='"tDocumentsFiles"' --custom-fetch-value=25

I don't particularly like the use of 'custom' in the name of the option,
seems like it's just a noise word and not really necessary.

>    I haven't tested the documentation: too many problems while building it
>    (also the original version, without my changes; probably I have bogus
>    tools... and too less time to check/try...).
>    Attached the patches for the master and REL9_6_STABLE.

I agree the documentation can be a bit of a pain, but there's a lot of
issues with the patch itself when it comes to the project style.  The
indentation doesn't look like it's all correct, and multi-line comments
should be of the form:

/** text here*/

Lastly, it'd be good to have this patch added to
https://commitfest.postgresql.org to have it formally reviewed in the
commitfest cycle coming up in January.

Thanks!

Stephen

pgsql-hackers by date:

Previous
From: Stephen Frost
Date:
Subject: Re: [HACKERS] [BUG?] pg_event_trigger_ddl_commands() error withALTER TEXT SEARCH CONFIGURATION
Next
From: Joe Conway
Date:
Subject: Re: [HACKERS] Getting rid of "unknown error" in dblink andpostgres_fdw