Re: Invalid YAML output from EXPLAIN

From: Dean Rasheed <dean(dot)a(dot)rasheed(at)gmail(dot)com>
To: Robert Haas <robertmhaas(at)gmail(dot)com>
Cc: pgsql-bugs(at)postgresql(dot)org
Subject: Re: Invalid YAML output from EXPLAIN
Date: 2010-06-09 11:23:03
Message-ID: AANLkTim1pKt_uN_5Oe043rmaHHcf9w50SER1R7C708-3@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs pgsql-hackers

On 9 June 2010 12:07, Robert Haas <robertmhaas(at)gmail(dot)com> wrote:
> On Wed, Jun 9, 2010 at 2:58 AM, Dean Rasheed <dean(dot)a(dot)rasheed(at)gmail(dot)com> wrote:
>> On 9 June 2010 03:48, Robert Haas <robertmhaas(at)gmail(dot)com> wrote:
>>> Er, I should also say, thanks for the report, and please test.  I am
>>> definitely not an expert on YAML.
>>>
>>
>> I'm not an expert on YAML either, but I don't think this works (at
>> least it breaks against the online YAML parser here:
>> http://yaml-online-parser.appspot.com/). If the string starts with a
>> ".", then it tries to treat it as a floating point number and baulks
>> if the rest of the string isn't a valid number.
>
> Really?  I enter:
>
> - foo
> - bar
> - .baz
>
> And it produces this JSON:
>
> [
>  "foo",
>  "bar",
>  ".baz"
> ]
>
> That looks OK to me.
>

Ah, OK I didn't test those cases properly before composing my email.
It's actually only a "." on its own that it can't parse.

- just: write some
- yaml:
- .

ERROR:

invalid literal for float(): .

I'm not sure if that's valid YAML or not.

My comment about numbers still applies though. The following are
different values:

- just: write some
- yaml:
- 123
- "123"

[
{
"just": "write some"
},
{
"yaml": [
123,
"123"
]
}
]

Regards,
Dean

In response to

Responses

Browse pgsql-bugs by date

  From Date Subject
Next Message Robert Haas 2010-06-09 11:32:29 Re: Invalid YAML output from EXPLAIN
Previous Message Joel Henrique 2010-06-09 11:15:21 Re: BUG #5475: Problem during Instalation

Browse pgsql-hackers by date

  From Date Subject
Next Message Robert Haas 2010-06-09 11:32:29 Re: Invalid YAML output from EXPLAIN
Previous Message Robert Haas 2010-06-09 11:09:24 Re: hstore ==> and deprecate =>