Skip site navigation (1) Skip section navigation (2)

Re: hstore parser incorrectly handles malformed input

From: Ryan Kelly <rpkelly22(at)gmail(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Vik Reykja <vikreykja(at)gmail(dot)com>, pgsql-bugs(at)postgresql(dot)org
Subject: Re: hstore parser incorrectly handles malformed input
Date: 2012-04-28 23:20:25
Message-ID: (view raw, whole thread or download thread mbox)
Lists: pgsql-bugs
On Fri, Apr 27, 2012 at 10:22:11AM -0400, Tom Lane wrote:
> Ryan Kelly <rpkelly22(at)gmail(dot)com> writes:
> > As long as we make it consistent on both sides of the '=>' (and document
> > it, too), then I don't really care either way. Currently you have to use
> > quotes to get an empty key, so I thought it natural to that you should
> > have to quote to get an empty value.
> > I've attached a modified version of Tom's patch which also allows empty
> > keys.
> Hm ... I don't agree that keys and values are interchangeable, and
> I don't see that empty keys are a good thing (whereas empty values are
> clearly a reasonable edge case).  So I think this is going a bit far;
> it seems to me it'd be giving up a lot of syntax-error detection
> capability in return for some not-actually-helpful symmetry.
I don't think any language supports the empty key edge case in this way.
The only language I know of that will let you get away with it is Perl,
and in that case you get undef, not the empty string, and it's a warning
if you have warnings on:

$ use warnings;
$ my $h = { 1 => };
Odd number of elements in anonymous hash
$HASH1 = { 1 => undef };

In fact, it's akin to doing:
[db]> select hstore('{foo}'::text[]);
ERROR:  array must have even number of elements

Or maybe even this:
[db]> select hstore('{foo,}'::text[]);
ERROR:  malformed array literal: "{foo,}"

>>> h = {1 : }
SyntaxError: invalid syntax

>> h = {1 => }
SyntaxError: compile error

csharp> var h = new Dictionary<String, String>(){ { "1", } };
{interactive}(1,50): error CS1525: Unexpected symbol `}'
csharp> var h = new Dictionary<String, String>(){ { "1" } };
{interactive}(1,44): error CS1501: No overload for method `Add' takes `1' arguments

js> a = { 1: }
typein:1: SyntaxError: syntax error:

$ echo '<?php $h = array( 1 => ); ?>' | php
PHP Parse error:  syntax error, unexpected ')'

> On the other hand, I seldom use hstore so I'm probably not the best
> person to be judging the appropriateness of these options.  Any other
> votes out there?
As long as we're voting, I vote for my original patch. I think requiring
something on the RHS of => to get a value is a sane and rather common thing.

> 			regards, tom lane

-Ryan Kelly

In response to


pgsql-bugs by date

Next:From: Alex HunsakerDate: 2012-04-28 23:52:20
Subject: Re: hstore parser incorrectly handles malformed input
Previous:From: Petteri R├ĄtyDate: 2012-04-28 22:45:37
Subject: Re: BUG #6619: Misleading output from slave when host is not running

Privacy Policy | About PostgreSQL
Copyright © 1996-2017 The PostgreSQL Global Development Group