[
https://issues.apache.org/jira/browse/S2GRAPH-53?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15175540#comment-15175540
]
ASF GitHub Bot commented on S2GRAPH-53:
---------------------------------------
GitHub user SteamShon opened a pull request:
https://github.com/apache/incubator-s2graph/pull/37
[S2GRAPH-53] Refactor Storage to decide which serializer/deserializer for
IndexEdge/SnapshotEdge/Vertex.
add serde package and change storage to contain compatability table.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/SteamShon/incubator-s2graph S2GRAPH-53
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-s2graph/pull/37.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #37
----
commit 73de01fae93c25ed192467d32c5de2fc7b4d4e55
Author: DO YUNG YOON <[email protected]>
Date: 2016-03-02T12:39:14Z
add serde package and change storage to contain compatability table.
----
> Refactor Storage to decide which serializer/deserializer for
> IndexEdge/SnapshotEdge/Vertex.
> -------------------------------------------------------------------------------------------
>
> Key: S2GRAPH-53
> URL: https://issues.apache.org/jira/browse/S2GRAPH-53
> Project: S2Graph
> Issue Type: Improvement
> Reporter: DOYUNG YOON
> Assignee: DOYUNG YOON
> Priority: Minor
> Labels: refactor, serde
> Original Estimate: 48h
> Remaining Estimate: 48h
>
> SerDe for IndexEdge/SnapshotEdge/Vertex works differently based on label's
> schemaVersion and Storage only use one SerDe pair for each ones now.
> Decision for which SerDe is used for each label should be controlled by
> Storage since some of storage implementation only use certain SerDe pair for
> every label schema version.
> The only reason we can't truncate schema v1, v2 is only for backward
> compatibility.
> I am suggesting moving logic to decide which SerDe when given label's
> schemaVersion from IndexEdge/SnapshotEdge SerDe to Storage so newly added
> storage implementation override this logic to avoid manually change test
> label`s schema version, since some of newly added storage implementation only
> support some of schema version.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
