Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commit99565a1

Browse files
committed
Make pg_dump behave more sanely when built without HAVE_LIBZ.
For some reason the code to emit a warning and switch to uncompressedoutput was placed down in the guts of pg_backup_archiver.c. This isdefinitely too late in the case of parallel operation (and I ratherwonder if it wasn't too late for other purposes as well). Put it inpg_dump.c's option-processing logic, which seems a much saner place.Also, the default behavior with custom or directory output format wasto emit the warning telling you the output would be uncompressed. Thisseems unhelpful, so silence that case.Back-patch to 9.3 where parallel dump was introduced.Kyotaro Horiguchi, adjusted a bit by meReport: <20160526.185551.242041780.horiguchi.kyotaro@lab.ntt.co.jp>
1 parentb9784e1 commit99565a1

File tree

2 files changed

+9
-10
lines changed

2 files changed

+9
-10
lines changed

‎src/bin/pg_dump/pg_backup_archiver.c

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -3214,17 +3214,7 @@ WriteHead(ArchiveHandle *AH)
32143214
(*AH->WriteBytePtr) (AH,AH->intSize);
32153215
(*AH->WriteBytePtr) (AH,AH->offSize);
32163216
(*AH->WriteBytePtr) (AH,AH->format);
3217-
3218-
#ifndefHAVE_LIBZ
3219-
if (AH->compression!=0)
3220-
write_msg(modulename,"WARNING: requested compression not available in this "
3221-
"installation -- archive will be uncompressed\n");
3222-
3223-
AH->compression=0;
3224-
#endif
3225-
32263217
WriteInt(AH,AH->compression);
3227-
32283218
crtm=*localtime(&AH->createDate);
32293219
WriteInt(AH,crtm.tm_sec);
32303220
WriteInt(AH,crtm.tm_min);

‎src/bin/pg_dump/pg_dump.c

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -595,12 +595,21 @@ main(int argc, char **argv)
595595
/* Custom and directory formats are compressed by default, others not */
596596
if (compressLevel==-1)
597597
{
598+
#ifdefHAVE_LIBZ
598599
if (archiveFormat==archCustom||archiveFormat==archDirectory)
599600
compressLevel=Z_DEFAULT_COMPRESSION;
600601
else
602+
#endif
601603
compressLevel=0;
602604
}
603605

606+
#ifndefHAVE_LIBZ
607+
if (compressLevel!=0)
608+
write_msg(NULL,"WARNING: requested compression not available in this "
609+
"installation -- archive will be uncompressed\n");
610+
compressLevel=0;
611+
#endif
612+
604613
/*
605614
* On Windows we can only have at most MAXIMUM_WAIT_OBJECTS (= 64 usually)
606615
* parallel jobs because that's the maximum limit for the

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp