[S2GRAPH-146] Merge two IndexEdgeDeserializable under tall, wide package into one.

• Type: Improvement
• Status: To Do
• Priority: Major
• Resolution: Unresolved
• Affects Version/s: 0.2.0
• Assignee: Do Yung Yoon
• Reporter: Do Yung Yoon
• Due: 13/Dec/16
• Created: 11/Dec/16 12:45
• Updated: 13/Dec/16 02:15

Description

Currently we have two IndexEdgeDeserializable class both for tall/wide schema.

I suggest to refactor these so we can just call one single IndexEdgeDeserializable without knowing which label we are traversing on.

Currently, caller should pass QueryParam which specify Label we are searching for, but to deal with search for all available Label get complicated with this restriction. If user want to fetch all edges stored, then things get complicated.

It would be better to refactor Deserializable to be able to know which Label we are deserializing while deserialize KeyValue from storage rather than depending on parameter passed from caller.

Issue Links

• links to: GitHub Pull Request #111