Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Skip to content

Commit f9f4b43

Browse files
committed
Address stats export review comments.
Per discussion, did not use Jian He's patch exactly. Reported-by: jian he <jian.universality@gmail.com> Reviewed-by: Corey Huinker <corey.huinker@gmail.com> Discussion: https://postgr.es/m/CACJufxFVq=tq9u1zrHWYSbMi1T07gS9Ff0LJScMco4HZmtZ1xw@mail.gmail.com Discussion: https://postgr.es/m/CADkLM=f1n2_Vomq0gKab7xdxDHmJGgn=DE48P8fzQOp3Mrs1Qg@mail.gmail.com
1 parent 298944e commit f9f4b43

File tree

1 file changed

+2
-6
lines changed

1 file changed

+2
-6
lines changed

src/bin/pg_dump/pg_backup_archiver.c

+2-6
Original file line numberDiff line numberDiff line change
@@ -2936,8 +2936,8 @@ _tocEntryRequired(TocEntry *te, teSection curSection, ArchiveHandle *AH)
29362936
{
29372937
if (!ropt->dumpStatistics)
29382938
return 0;
2939-
else
2940-
res = REQ_STATS;
2939+
2940+
res = REQ_STATS;
29412941
}
29422942

29432943
/*
@@ -2984,10 +2984,6 @@ _tocEntryRequired(TocEntry *te, teSection curSection, ArchiveHandle *AH)
29842984
if (ropt->no_subscriptions && strcmp(te->desc, "SUBSCRIPTION") == 0)
29852985
return 0;
29862986

2987-
/* If it's statistics and we don't want statistics, maybe ignore it */
2988-
if (!ropt->dumpStatistics && strcmp(te->desc, "STATISTICS DATA") == 0)
2989-
return 0;
2990-
29912987
/* Ignore it if section is not to be dumped/restored */
29922988
switch (curSection)
29932989
{

0 commit comments

Comments
 (0)