geocouch support

[Darren Grant]

Hello,

Would it be possible to add support for geocouch?

A quick test seems to show the only changes needed are as follows:

- a parameter to the Database.view() method to change '_view' to '_spatial' (client.py:778).

- change to Row.__repr__() as there is no 'key' in the result

Alternatively I can submit a patch if someone can show me the ropes..

Thank you,

Darren

[Dirkjan Ochtman]

On Fri, Nov 11, 2011 at 06:44, Darren Grant <darren....@gmail.com> wrote:
> A quick test seems to show the only changes needed are as follows:
> - a parameter to the Database.view() method to change '_view' to '_spatial'
> (client.py:778).
> - change to Row.__repr__() as there is no 'key' in the result
> Alternatively I can submit a patch if someone can show me the ropes..

As I don't have a geocouch to test with, a patch would help. But based
on the described changes, how will you make the interface work with
both normal views and geoviews? Also, you should add a test that's
conditional on geocouch being available.

As for the ropes, I think
http://code.google.com/p/couchdb-python/source/checkout tells you
how to get source. It's probably easiest to just create an issue and
attach your patch there.

Cheers,

Dirkjan

[Darren Grant]

Sure, also there should be tests written for it..

Actually we should probably link up with the couchbase team to make sure we're being consistent with the bigger picture (e.g. can there be other _x namespaces we should take into account).  I'll investigate a bit more, then send an email and copy you in.