]> git.ipfire.org Git - thirdparty/postgresql.git/commitdiff
Fix broken XML formatting in EXPLAIN output for incremental sorts.
authorTom Lane <tgl@sss.pgh.pa.us>
Fri, 23 Oct 2020 15:32:33 +0000 (11:32 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Fri, 23 Oct 2020 15:32:33 +0000 (11:32 -0400)
The ExplainCloseGroup arguments for incremental sort usage data
didn't match the corresponding ExplainOpenGroup.  This only matters
for XML-format output, which is probably why we'd not noticed.

Daniel Gustafsson, per bug #16683 from Frits Jalvingh

Discussion: https://postgr.es/m/16683-8005033324ad34e9@postgresql.org

src/backend/commands/explain.c

index c98c9b5547c5a5a08fa98040f239bb13a85a54ae..0ad49612d28c7a31deec21474259326890900aa7 100644 (file)
@@ -2830,7 +2830,7 @@ show_incremental_sort_group_info(IncrementalSortGroupInfo *groupInfo,
                        ExplainPropertyInteger("Peak Sort Space Used", "kB",
                                                                   groupInfo->maxMemorySpaceUsed, es);
 
-                       ExplainCloseGroup("Sort Spaces", memoryName.data, true, es);
+                       ExplainCloseGroup("Sort Space", memoryName.data, true, es);
                }
                if (groupInfo->maxDiskSpaceUsed > 0)
                {
@@ -2847,7 +2847,7 @@ show_incremental_sort_group_info(IncrementalSortGroupInfo *groupInfo,
                        ExplainPropertyInteger("Peak Sort Space Used", "kB",
                                                                   groupInfo->maxDiskSpaceUsed, es);
 
-                       ExplainCloseGroup("Sort Spaces", diskName.data, true, es);
+                       ExplainCloseGroup("Sort Space", diskName.data, true, es);
                }
 
                ExplainCloseGroup("Incremental Sort Groups", groupName.data, true, es);