Re: json api WIP patch - Mailing list pgsql-hackers

From David Fetter
Subject Re: json api WIP patch
Date
Msg-id 20130115190424.GB32407@fetter.org
Whole thread Raw
In response to Re: json api WIP patch  (Andrew Dunstan <andrew@dunslane.net>)
Responses Re: json api WIP patch
Re: json api WIP patch
List pgsql-hackers
On Mon, Jan 14, 2013 at 07:52:56PM -0500, Andrew Dunstan wrote:
> 
> On 01/14/2013 07:36 PM, Merlin Moncure wrote:
> >While testing this I noticed that integer based 'get' routines are
> >zero based -- was this intentional?  Virtually all other aspects of
> >SQL are 1 based:
> >
> >postgres=# select json_get('[1,2,3]', 1);
> >  json_get
> >----------
> >  2
> >(1 row)
> >
> >postgres=# select json_get('[1,2,3]', 0);
> >  json_get
> >----------
> >  1
> >(1 row)
> 
> Yes. it's intentional. SQL arrays might be 1-based by default, but
> JavaScript arrays are not. JsonPath and similar gadgets treat the
> arrays as zero-based. I suspect the Json-using community would not
> thank us for being overly SQL-centric on this - and I say that as
> someone who has always thought zero based arrays were a major design
> mistake, responsible for countless off-by-one errors.

Perhaps we could compromise by making arrays 0.5-based.

Cheers,
David.
-- 
David Fetter <david@fetter.org> http://fetter.org/
Phone: +1 415 235 3778  AIM: dfetter666  Yahoo!: dfetter
Skype: davidfetter      XMPP: david.fetter@gmail.com
iCal: webcal://www.tripit.com/feed/ical/people/david74/tripit.ics

Remember to vote!
Consider donating to Postgres: http://www.postgresql.org/about/donate



pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: Patches for TODO item: Avoid truncating empty OCDR temp tables on COMMIT
Next
From: Andres Freund
Date:
Subject: Re: unlogged tables vs. GIST