aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTom Lane <tgl@sss.pgh.pa.us>2011-09-25 13:59:17 -0400
committerTom Lane <tgl@sss.pgh.pa.us>2011-09-25 13:59:17 -0400
commit23fe7a74777eba01835389263418cbe8a546e772 (patch)
tree4b377def6084549bd8a3fd1e0201b75f56cd7f19
parent84bbe57dc128df224614c61f88c350ee9ed92123 (diff)
downloadpostgresql-23fe7a74777eba01835389263418cbe8a546e772.tar.gz
postgresql-23fe7a74777eba01835389263418cbe8a546e772.zip
Un-break compression of plain-text output format in pg_dump.
pg_dump has historically understood -Z with no -F switch to mean that it should emit a gzip-compressed version of its plain text output. This got broken through a misunderstanding in the 9.1 patch that added directory output format. Restore the former behavior. Per complaint from Roger Niederland and diagnosis by Adrian Klaver.
-rw-r--r--src/bin/pg_dump/pg_dump.c12
1 files changed, 3 insertions, 9 deletions
diff --git a/src/bin/pg_dump/pg_dump.c b/src/bin/pg_dump/pg_dump.c
index 56db6e54c14..b4ab19d5e05 100644
--- a/src/bin/pg_dump/pg_dump.c
+++ b/src/bin/pg_dump/pg_dump.c
@@ -528,20 +528,14 @@ main(int argc, char **argv)
exit(1);
}
+ /* Identify archive format to emit */
archiveFormat = parseArchiveFormat(format, &archiveMode);
/* archiveFormat specific setup */
if (archiveFormat == archNull)
plainText = 1;
- /*
- * Ignore compression level for plain format. XXX: This is a bit
- * inconsistent, tar-format throws an error instead.
- */
- if (archiveFormat == archNull)
- compressLevel = 0;
-
- /* Custom and directory formats are compressed by default */
+ /* Custom and directory formats are compressed by default, others not */
if (compressLevel == -1)
{
if (archiveFormat == archCustom || archiveFormat == archDirectory)
@@ -550,7 +544,7 @@ main(int argc, char **argv)
compressLevel = 0;
}
- /* open the output file */
+ /* Open the output file */
g_fout = CreateArchive(filename, archiveFormat, compressLevel, archiveMode);
if (g_fout == NULL)