Fix broken XML formatting in EXPLAIN output for incremental sorts.
authorTom Lane
Fri, 23 Oct 2020 15:32:33 +0000 (11:32 -0400)
committerTom Lane
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);