[TINKERPOP-2070] gremlin-javascript: Introduce Connection representation

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Done
• Affects Version/s: 3.3.4
• Fix Version/s: 3.4.0, 3.3.5
• Component/s: javascript
• Assignee: Jorge Bay
• Reporter: Jorge Bay
• Created: 17/Oct/18 13:38
• Updated: 05/Nov/18 13:31
• Resolved: 05/Nov/18 13:31

Description

Currently on gremlin-javascript, DriverRemoteConnection represents a single connection to the server and a RemoteConnection implementation.

Like Python and .NET, we need to make the DriverRemoteConnection use the Client instead of interacting with the websocket directly.

The dependency tree between classes should be like the following

DriverRemoteConnection
|_ Client
  |_ Connection

These changes don't involve a breaking change, its more of an implementation issue. As DriverRemoteConnection is part of the public API the current implementation is a limitation.

This blocks or at least forces awful workarounds for improvements like TINKERPOP-2064.

 

 

Issue Links

• blocks: TINKERPOP-2064 Add status attributes to results for gremlin-javascript
• links to: GitHub Pull Request #966