Make pg_dump behave more sanely when built without HAVE_LIBZ.
authorTom Lane <tgl@sss.pgh.pa.us>
Thu, 26 May 2016 15:51:04 +0000 (11:51 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Thu, 26 May 2016 15:51:11 +0000 (11:51 -0400)
commit64b296976befff7ec0debdc82db96b7dd3c7f45a
tree519b5883521d7a91d64a47e847c636cf010ddebd
parent6479df1378607e5edbe19cc28a3b52c62f11d8fa
Make pg_dump behave more sanely when built without HAVE_LIBZ.

For some reason the code to emit a warning and switch to uncompressed
output was placed down in the guts of pg_backup_archiver.c.  This is
definitely too late in the case of parallel operation (and I rather
wonder if it wasn't too late for other purposes as well).  Put it in
pg_dump.c's option-processing logic, which seems a much saner place.

Also, the default behavior with custom or directory output format was
to emit the warning telling you the output would be uncompressed.  This
seems unhelpful, so silence that case.

Back-patch to 9.3 where parallel dump was introduced.

Kyotaro Horiguchi, adjusted a bit by me

Report: <20160526.185551.242041780.horiguchi.kyotaro@lab.ntt.co.jp>
src/bin/pg_dump/pg_backup_archiver.c
src/bin/pg_dump/pg_dump.c