DSE Graph integration

Description

Add support for DSE Graph in the Python driver API.

Environment

None

Pull Requests

None

Activity

Show:
Adam Holmberg
January 9, 2016, 2:43 AM

https://github.com/datastax/python-driver/pull/458

PR presently just wraps a Session in a GraphSession. We're still discussing the merits of wrapping vs extending. Some discussion here: https://github.com/datastax/java-driver-dse/pull/3

Many of the points for wrapping seem to be java-specific. I wanted to put this up for review to start a discussion focused on Python. I was leaning toward adding graph execute methods to the specialized session created for the geo ticket.

Michael Penick
January 15, 2016, 7:42 AM

Looks good to me @ccae3183ea69b4e8cd8beebe91e88393970a6ee2. "GraphBoundStatement" is mentioned in one of the comments but not implemented.

Michael Penick
January 15, 2016, 7:50 AM

I like the compostability of wrapping versus extending. Wrapping allows a single session it to be used in multiple ways.

Adam Holmberg
January 16, 2016, 8:23 AM

Extending still allows it to be used for cql and graph. I won't expound too much more as there are ongoing debates elsewhere.

For now, merging with wrapped approach.

Greg Bestland
February 4, 2016, 2:07 PM

I'm moving this to done, with the understanding that the tests are not running reliably in Jenkins, but from a functional standpoint it's tested.

Fixed

Assignee

Unassigned

Reporter

Kevin Gallardo

Fix versions

Labels

PM Priority

None

External issue ID

None

Doc Impact

None

Reviewer

None

Size

None

Pull Request

None

Components

Sprint

Py P-NEXT

Priority

Major