search

jkschneider / pappus-graph-projection

Map any Java object model to a Tinkerpop compatible graph!
Apache License 2.0
14 stars 0 forks source link

Request to refactor code to allow for GenericMapReader/Writer similar to GraphsonReader/Writer #5

Open ramukima opened 10 years ago

ramukima commented 10 years ago

Would it make sense to refactor the code so that we extract any JSON data structure can be serialized/deserialized to/from TinkerGraph implementation ?

jkschneider commented 10 years ago

Do you mean that (1) you want to be able to retrieve the intermediary Map that is created as part of the serialization and deserialization or (2) that you would prefer the intermediary format to be GraphSON?

ramukima commented 10 years ago

This issue is dependent on a fix for issue#4. A fix to issue#4 will allow me to dump any JSON data structure (not necessarily GraphSon compliant) into a graph database as long as I can convert the JSON into a Map<String, Object> somehow.

Given that issue#4 is resolved, it would be useful to refactor part of the code and create something which can read/write RAW JSON into a graph, using this library. May be name space refactoring, and just renaming classes along the lines of *Reader/Writer libraries from tinkerpop io module, is what I am thinking of.

Feel free to close this issue if you do not find it useful to do the refactoring. A fix to issue#4 is definitely going to be useful (even a configurable behavior is fine).