| From: | rhaas(at)postgresql(dot)org (Robert Haas) |
|---|---|
| To: | pgsql-committers(at)postgresql(dot)org |
| Subject: | pgsql: Quote all string values in EXPLAIN (FORMAT YAML) output. |
| Date: | 2010-06-10 01:26:30 |
| Message-ID: | 20100610012630.C628D7541D4@cvs.postgresql.org |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-committers |
Log Message:
-----------
Quote all string values in EXPLAIN (FORMAT YAML) output.
While my previous attempt seems to always produce valid YAML, it
doesn't always produce YAML that means what it appears to mean,
because of tokens like "0xa" and "true", which without quotes will
be interpreted as integer or Boolean literals. So, instead, just
quote everything that's not known to be a number, as we do for
JSON.
Dean Rasheed, with some changes to the comments by me.
Modified Files:
--------------
pgsql/src/backend/commands:
explain.c (r1.205 -> r1.206)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/commands/explain.c?r1=1.205&r2=1.206)
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Tom Lane | 2010-06-10 04:05:01 | pgsql: Fix quite-bogus handling of arrays in plpython datum-to-PyObject |
| Previous Message | Takahiro Itagaki | 2010-06-10 00:48:51 | Re: pgsql: Fix connection leak in dblink when dblink_connect() or |