[S2GRAPH-17] Remove unnecessary abstraction layer, Storage.

• Type: Task
• Status: Done
• Priority: Major
• Resolution: Done
• Labels: client refactoring
• Assignee: Do Yung Yoon
• Reporter: Do Yung Yoon
• Due: 22/Jan/16
• Created: 05/Jan/16 23:23
• Updated: 09/Jun/16 09:52
• Resolved: 09/Jun/16 09:52

Description

Currently Graph contains following components.

Graph has storage and each storage have followings.

1. SerDe: serialize and deserialize IndexEdge/SnapshotEdge/Vertex into bytes.
2. MutationBuilder: convert IndexEdge/SnapshotEdge into RPC which is dependent on storage backend.
3. QueryBuilder: issue RPC for fetch on storage backend

With these components, storage know how to actually store/fetch IndexEdge/SnapshotEdge/Vertex to, from storage backend.

In my opinion, storage layer is unnecessary since all graph instance does is delegating into storage. I am suggesting to remove storage layer and use different Graph class for different storage backend.

So for example, change AsynchbaseStorage into AsynchbaseGraph.

I want to discuss how our Graph Client should be on this issue and change it according to discussion.