Thread: BUG #12730: pg_dump produces trailing spaces
The following bug has been logged on the website: Bug reference: 12730 Logged by: Hans Ginzel Email address: hans@matfyz.cz PostgreSQL version: 9.4.0 Operating system: Centos 6.5 Description: Hello! Try pg_dump --format=p --schema=information_schema --schema-only --file=is.sql It seams there are trailing spaces in the comment line -- Name: sql_implementation_info; Type: TABLE; Schema: information_schema; Owner: postgres; Tablespace: Skip the string '; Tablespace: ' please, if there is no tablespace or put '-' for the tablespace name in the same way as dash is used for Owner when the option --no-owner is used. Regards Hans
On Mon, Feb 2, 2015 at 02:31:16PM +0000, hans@matfyz.cz wrote: > The following bug has been logged on the website: > > Bug reference: 12730 > Logged by: Hans Ginzel > Email address: hans@matfyz.cz > PostgreSQL version: 9.4.0 > Operating system: Centos 6.5 > Description: > > Hello! > > Try > pg_dump --format=p --schema=information_schema --schema-only --file=is.sql > > It seams there are trailing spaces in the comment line > -- Name: sql_implementation_info; Type: TABLE; Schema: information_schema; > Owner: postgres; Tablespace: > > Skip the string '; Tablespace: ' please, if there is no tablespace or put > '-' for the tablespace name in the same way as dash is used for Owner when > the option --no-owner is used. I looked into this issue and found that the pg_dump code doesn't handle cases where the reltablespace == 0 (default). I have modified pg_dump to test for a zero-length string as well, and this suppresses the "Tablespace:" field. Patch attached. -- Bruce Momjian <bruce@momjian.us> http://momjian.us EnterpriseDB http://enterprisedb.com + Everyone has their own god. +
Attachment
On Tue, Apr 28, 2015 at 07:40:18PM -0400, Bruce Momjian wrote: > > Try > > pg_dump --format=p --schema=information_schema --schema-only --file=is.sql > > > > It seams there are trailing spaces in the comment line > > -- Name: sql_implementation_info; Type: TABLE; Schema: information_schema; > > Owner: postgres; Tablespace: > > > > Skip the string '; Tablespace: ' please, if there is no tablespace or put > > '-' for the tablespace name in the same way as dash is used for Owner when > > the option --no-owner is used. > > I looked into this issue and found that the pg_dump code doesn't handle > cases where the reltablespace == 0 (default). I have modified pg_dump > to test for a zero-length string as well, and this suppresses the > "Tablespace:" field. Patch attached. Patch applied. -- Bruce Momjian <bruce@momjian.us> http://momjian.us EnterpriseDB http://enterprisedb.com + Everyone has their own god. +