[jira] [Created] (S2GRAPH-121) Create `Result` class to hold traverse result edges.

["DOYUNG YOON (JIRA)" <ji...@apache.org>]

DOYUNG YOON created S2GRAPH-121:
-----------------------------------

             Summary: Create `Result` class to hold traverse result edges.
                 Key: S2GRAPH-121
                 URL: https://issues.apache.org/jira/browse/S2GRAPH-121
             Project: S2Graph
          Issue Type: Improvement
    Affects Versions: 0.2.0
            Reporter: DOYUNG YOON
            Assignee: DOYUNG YOON
             Fix For: 0.2.0


Purpose of suggestion.

GroupBy, OrderBy, Filtering implementation of fetched result edges are spread out multiple parts(Storage, AsynchbaseStorage, PostProcess). It would be much better we can separate post process logic(OrderBy, GroupBy, FilterOut) in `Result` class so any caller simply can benefit single implementation.

In general, I am suggesting changing PostProcess only call method of Result class, not actually implement logic.

I was asked if we can provide binary format(not JSON) such as protobuf, thrift, etc and I think it is reasonable demand. While I was tapping out possibility of this, I found out moving post process implementation into `Result` class can make this really easy, I guess we can simply focus on actual `format` not post process logic itself.




--
This message was sent by Atlassian JIRA
(v6.3.4#6332)