[S2GRAPH-170] Create Interface for S2Edge/S2Vertex/S2Graph.

• Type: Improvement
• Status: Done
• Priority: Major
• Resolution: Done
• Affects Version/s: 0.2.1
• Fix Version/s: 0.2.1
• Assignee: Do Yung Yoon
• Reporter: Do Yung Yoon
• Due: 30/Nov/17
• Created: 03/Nov/17 22:37
• Updated: 21/Nov/17 00:04
• Resolved: 21/Nov/17 00:04

Description

Problem Statement

S2Graph’s entire code base is dependent on S2Edge/S2Vertex/S2Graph class. Even though lots of code touch theses two class, there is no interface defined currently. This means lots of code is interact with these class in different way all by their own way, and this make extremely hard to make any change on these two classes.

For example, I was working on S2GRAPH-80 to provide java client, and there are too many places to be changed since all theses places use concrete implementation class S2Edge/S2Vertex/S2Graph, not the interfaces. Not just for S2GRAPH-80, but any other issues that need to change theses classes would benefit by communicating through interface.

Suggestion

Define interface and change code base to communicate through theses interfaces.

1. Create interface S2Edge/S2Vertex/S2Graph that implement Tinkerpop’s Edge/Vertex/Graph interface
2. Extract tinkerpop interface related implementations
3. Change caller of S2Edge/S2Vertex/S2Graph to call interface rather than concrete implementation.

Issue Links

• links to: GitHub Pull Request #127