Re: machine-readable explain output v4

From: Andrew Dunstan <andrew(at)dunslane(dot)net>
To: Andrew Dunstan <andrew(at)dunslane(dot)net>
Cc: Andres Freund <andres(at)anarazel(dot)de>, 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:48:29
Message-ID: 4A7F6E5D.2040703@dunslane.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Andrew Dunstan wrote:
>
>
> 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.

I takle it back. It's still there at
<http://archives.postgresql.org/pgsql-hackers/2009-08/msg00485.php>
posted 3 days ago.

cheers

andrew

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2009-08-10 00:53:16 Re: machine-readable explain output v4
Previous Message Andrew Dunstan 2009-08-10 00:42:36 Re: machine-readable explain output v4