[ https://issues.apache.org/jira/browse/TINKERPOP-2006?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16563604#comment-16563604 ]
stephen mallette commented on TINKERPOP-2006: --------------------------------------------- regarding (4) - I think it should go here: https://github.com/apache/tinkerpop/blob/e3018fb2e98df62a21272446f63cea5ee550408e/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/structure/io/IoTest.java#L108 Since it is a one-off test just for GraphML, I think you can just embed a small GraphML file as a {{String}} right in the test. [~twilmes] please note that the comments related to your comments in github are in this JIRA. > GraphML serialization invalid if a vertex and edge have similar named property > ------------------------------------------------------------------------------ > > Key: TINKERPOP-2006 > URL: https://issues.apache.org/jira/browse/TINKERPOP-2006 > Project: TinkerPop > Issue Type: Bug > Components: io > Affects Versions: 3.3.3 > Reporter: Svante Schubert > Priority: Trivial > > I have created a graph using the Tinkerpop Graph and required for Gephi > visualization a property called color on edges and vertices to be coloured. > The current gremlin-core serialization creates the following two key elements > in GraphML > <key id="color" for="node" attr.name="color" attr.type="string"></key> > <key id="color" for="edge" attr.name="color" attr.type="string"></key> > the id attribute is an internal name, but have to be different. > I would suggest a patch to check for an intersection of the edge & vertices > keys and add for those keys an additional differentiating letter like: > <key id="colorV" for="node" attr.name="color" attr.type="string"></key> > <key id="colorE" for="edge" attr.name="color" attr.type="string"></key> > > Going to provide a pullrequest on GitHub. -- This message was sent by Atlassian JIRA (v7.6.3#76005)