You are viewing a plain text version of this content. The canonical link for it is here.
Posted to user@couchdb.apache.org by Chris Anderson <jc...@apache.org> on 2009/07/25 21:42:02 UTC

JSON (was Re: CouchDB / Mochiweb version)

On Sat, Jul 25, 2009 at 12:29 PM, Paul Davis<pa...@gmail.com> wrote:
> On Sat, Jul 25, 2009 at 9:54 AM, Lynton
> Grice<ly...@logosworld.com> wrote:
>> Hi there,
>>
>> Can someone let me know why CouchDB does not work with the latest version of
>> Mochiweb from the web? I get JSON encoding errors using the latest version
>> of Mochiweb. All the test suite tests fail with syntax errors. What version
>> of Mochiweb is bundled with the latest version of CouchDB?
>>
>> There seems to be a difference in the "mochijson2.erl" file between the
>> mochiweb file bundled with the CouchDB download vs the latest Mochiweb "
>> mochijson2.erl".
>>
>> Any ideas?
>>
>> Lynton
>>
>>
>
> CouchDB's JSON serialization has always been slightly different than
> mochiweb's. At one point there was supposed to be a shift towards what
> CouchDB uses. There's a thread somewhere on erlang-questions IIRC that
> this move was discussed.
>
> HTH,
> Paul Davis
>

Moving this over to dev@ - although user@ is still on this reply.

Maybe we should just rename our patched version of the mochijson2
library to make the changes clear.

couch_json or couch_mochijson both come to mind.

Maybe changing names is silly - it depends on how many people are out
there having the same issue as Lynton. It would certainly make the
fact that it's patched more clear.

Also I'm starting to think we'll need someone to push the json bifs
stuff forward with the OTP team. It's not gonna just make itself
happen. They say there is going to be an easier practice for native
extensions, but perhaps it will take the imminent pressure of
encode_json and decode_json to light a fire under that effort.

Thoughts? This probably doesn't matter for 0.10

Chris

-- 
Chris Anderson
http://jchrisa.net
http://couch.io

RE: JSON (was Re: CouchDB / Mochiweb version)

Posted by Lynton Grice <ly...@logosworld.com>.
Hi Chris,

I would say rename the "CouchDB version of Mochiweb" to something else.
Obviously in a perfect world it would be great for the CouchDB Mochiweb and
the "separate Mochiweb" to be perfectly in sync, but if not then I suppose
renaming is an option.

Also, perhaps when compiling CouchDB maybe you should compile the "CouchDB
Mochiweb" into an "internal folder" of the CouchDB folder, rather than
compiling into the EBIN of a "Mochiweb folder" under "/lib". For instance in
my case I already had Mochiweb installed under "/lib" and then I compiled
CouchDB which confused things "Mochiweb" wise.

Looking forward to seeing what happens on this, I really enjoy CouchDB!

Thanks

Lynton 

-----Original Message-----
From: jchris@gmail.com [mailto:jchris@gmail.com] On Behalf Of Chris Anderson
Sent: 25 July 2009 09:42 PM
To: dev@couchdb.apache.org; user@couchdb.apache.org
Subject: JSON (was Re: CouchDB / Mochiweb version)

On Sat, Jul 25, 2009 at 12:29 PM, Paul Davis<pa...@gmail.com>
wrote:
> On Sat, Jul 25, 2009 at 9:54 AM, Lynton
> Grice<ly...@logosworld.com> wrote:
>> Hi there,
>>
>> Can someone let me know why CouchDB does not work with the latest version
of
>> Mochiweb from the web? I get JSON encoding errors using the latest
version
>> of Mochiweb. All the test suite tests fail with syntax errors. What
version
>> of Mochiweb is bundled with the latest version of CouchDB?
>>
>> There seems to be a difference in the "mochijson2.erl" file between the
>> mochiweb file bundled with the CouchDB download vs the latest Mochiweb "
>> mochijson2.erl".
>>
>> Any ideas?
>>
>> Lynton
>>
>>
>
> CouchDB's JSON serialization has always been slightly different than
> mochiweb's. At one point there was supposed to be a shift towards what
> CouchDB uses. There's a thread somewhere on erlang-questions IIRC that
> this move was discussed.
>
> HTH,
> Paul Davis
>

Moving this over to dev@ - although user@ is still on this reply.

Maybe we should just rename our patched version of the mochijson2
library to make the changes clear.

couch_json or couch_mochijson both come to mind.

Maybe changing names is silly - it depends on how many people are out
there having the same issue as Lynton. It would certainly make the
fact that it's patched more clear.

Also I'm starting to think we'll need someone to push the json bifs
stuff forward with the OTP team. It's not gonna just make itself
happen. They say there is going to be an easier practice for native
extensions, but perhaps it will take the imminent pressure of
encode_json and decode_json to light a fire under that effort.

Thoughts? This probably doesn't matter for 0.10

Chris

-- 
Chris Anderson
http://jchrisa.net
http://couch.io


Re: JSON (was Re: CouchDB / Mochiweb version)

Posted by Paul Davis <pa...@gmail.com>.
On Sun, Jul 26, 2009 at 2:58 AM, Lynton
Grice<ly...@logosworld.com> wrote:
> Hi Chris,
>
> I would say rename the "CouchDB version of Mochiweb" to something else.
> Obviously in a perfect world it would be great for the CouchDB Mochiweb and
> the "separate Mochiweb" to be perfectly in sync, but if not then I suppose
> renaming is an option.
>
> Also, perhaps when compiling CouchDB maybe you should compile the "CouchDB
> Mochiweb" into an "internal folder" of the CouchDB folder, rather than
> compiling into the EBIN of a "Mochiweb folder" under "/lib". For instance in
> my case I already had Mochiweb installed under "/lib" and then I compiled
> CouchDB which confused things "Mochiweb" wise.
>

Doesn't mochiweb get installed in $PREFIX/lib/couchdb/mochiweb-version/ ?

> Looking forward to seeing what happens on this, I really enjoy CouchDB!
>
> Thanks
>
> Lynton
>
> -----Original Message-----
> From: jchris@gmail.com [mailto:jchris@gmail.com] On Behalf Of Chris Anderson
> Sent: 25 July 2009 09:42 PM
> To: dev@couchdb.apache.org; user@couchdb.apache.org
> Subject: JSON (was Re: CouchDB / Mochiweb version)
>
> On Sat, Jul 25, 2009 at 12:29 PM, Paul Davis<pa...@gmail.com>
> wrote:
>> On Sat, Jul 25, 2009 at 9:54 AM, Lynton
>> Grice<ly...@logosworld.com> wrote:
>>> Hi there,
>>>
>>> Can someone let me know why CouchDB does not work with the latest version
> of
>>> Mochiweb from the web? I get JSON encoding errors using the latest
> version
>>> of Mochiweb. All the test suite tests fail with syntax errors. What
> version
>>> of Mochiweb is bundled with the latest version of CouchDB?
>>>
>>> There seems to be a difference in the "mochijson2.erl" file between the
>>> mochiweb file bundled with the CouchDB download vs the latest Mochiweb "
>>> mochijson2.erl".
>>>
>>> Any ideas?
>>>
>>> Lynton
>>>
>>>
>>
>> CouchDB's JSON serialization has always been slightly different than
>> mochiweb's. At one point there was supposed to be a shift towards what
>> CouchDB uses. There's a thread somewhere on erlang-questions IIRC that
>> this move was discussed.
>>
>> HTH,
>> Paul Davis
>>
>
> Moving this over to dev@ - although user@ is still on this reply.
>
> Maybe we should just rename our patched version of the mochijson2
> library to make the changes clear.
>
> couch_json or couch_mochijson both come to mind.
>
> Maybe changing names is silly - it depends on how many people are out
> there having the same issue as Lynton. It would certainly make the
> fact that it's patched more clear.
>
> Also I'm starting to think we'll need someone to push the json bifs
> stuff forward with the OTP team. It's not gonna just make itself
> happen. They say there is going to be an easier practice for native
> extensions, but perhaps it will take the imminent pressure of
> encode_json and decode_json to light a fire under that effort.
>
> Thoughts? This probably doesn't matter for 0.10
>
> Chris
>
> --
> Chris Anderson
> http://jchrisa.net
> http://couch.io
>
>