Pull request to address one connection per send on DefaultClient

[avinash240]

The session based client is faster than the default one, especially at high rates. It's no longer paying the connection and breakdown penalty per socket and instead is connection pooling, which is great since the current DefaultArangoAgent was creating a new connection per document sent.
There are two tests, the geo tests. They don't return back the unique flag that you send in, I suspect that that index type does not support unique. However, I'm new to arangodb, so I don't know perhaps it's a bug in the server I'm running 2.5.5. I also added the sparse and selectivity keys where needed. I'm not sure if this breaks backwards compat with older servers, I'm sorry if it does, I just didn't want to hand back a broken test suite especially since you went through the trouble to build one.

[joowani]

Hello Marlon,

I've updated py-arango driver to support version 2.5.

Thanks!

Sincerely,
Joohwan

On Tue, Jun 9, 2015 at 8:25 AM, Marlon Bailey notifications@github.com
wrote:

The session based client is faster than the default one, especially at
high rates. It's no longer paying the connection and breakdown penalty per
socket and instead is connection pooling, which is great since the current
DefaultArangoAgent was creating a new connection per document sent.
There are two tests, the geo tests. They don't return back the unique flag
that you send in, I suspect that that index type does not support unique.
However, I'm new to arangodb, so I don't know perhaps it's a bug in the
server I'm running 2.5.5. I also added the sparse and selectivity keys
where needed. I'm not sure if this breaks backwards compat with older
servers, I'm sorry if it does, I just didn't want to hand back a broken

test suite especially since you went through the trouble to build one.

You can view, comment on, or merge this pull request online at:

#3
Commit Summary

• changed name to keep similar naming convention
• fixed the tests I can, skipped the ones I don't want to make
  assumptions on, added close method to sessionbased client, and added
  session_based boolean method to base class so code can be written to call
  close if necessary for cleanup

File Changes

• M arango/api.py
  https://github.com/Joowani/py-arango/pull/3/files#diff-0 (2)
• M arango/clients/base.py
  https://github.com/Joowani/py-arango/pull/3/files#diff-1 (3)
• A arango/clients/session.py
  https://github.com/Joowani/py-arango/pull/3/files#diff-2 (71)
• M arango/tests/test_index.py
  https://github.com/Joowani/py-arango/pull/3/files#diff-3 (24)
• M docs/arango.clients.rst
  https://github.com/Joowani/py-arango/pull/3/files#diff-4 (8)

Patch Links:

• https://github.com/Joowani/py-arango/pull/3.patch
• https://github.com/Joowani/py-arango/pull/3.diff

—
Reply to this email directly or view it on GitHub
#3.