Re: machine-readable explain output v4

From: Andrew Dunstan <andrew(at)dunslane(dot)net>
To: Andres Freund <andres(at)anarazel(dot)de>
Cc: pgsql-hackers(at)postgresql(dot)org, Robert Haas <robertmhaas(at)gmail(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Subject: Re: machine-readable explain output v4
Date: 2009-08-10 00:42:36
Message-ID: 4A7F6CFC.20403@dunslane.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Andres Freund wrote:
>> BTW, has anyone tried validating the XML at all? I just looked very
>> briefly at the patch at
>> <http://archives.postgresql.org/pgsql-hackers/2009-07/msg01944.php> and
>> I noticed this which makes me suspicious:
>>
>> + if (es.format == EXPLAIN_FORMAT_XML)
>> + appendStringInfoString(es.str,
>> + "<explain xmlns=\"http://www.postgresql.org/2009/explain\"
>> <http://www.postgresql.org/2009/explain%5C%22>;>\n");
>>
> That bug is fixed - as referenced above I wrote a schema and validated it. So,
> yes, the generated XML was valid at least before the last round of
> refactoring. And I looked through the output quite a bit so I would surprised
> if there is such a breakage.
> then someone needs to update the commitfest page with the lastest patch. That's the link I followed.
>
>
>
>

Hmm. I wonder how i got the link to an old version of the patch.

Anyway, I'm glad it's fixed.

cheers

andrew

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Andrew Dunstan 2009-08-10 00:48:29 Re: machine-readable explain output v4
Previous Message Andres Freund 2009-08-10 00:08:28 Re: machine-readable explain output v4