Github user spmallette commented on the issue:

    https://github.com/apache/tinkerpop/pull/892
  
    > at least reachable by API via Gremlin console for GraphML serialization
    
    It's reachable - just configured the `GraphWriter` 
[directly](http://tinkerpop.apache.org/docs/current/reference/#_graphml_reader_writer)
 rather than rely on the default provided by `graph.io(graphml())`
    
    ```java
    final Graph graph = TinkerFactory.createModern();
    try (final OutputStream os = new FileOutputStream("tinkerpop-modern.xml")) {
        
graph.io(IoCore.graphml()).writer().normalize(true).create().writeGraph(os, 
graph);
    }
    ```
    
    > Again, if you do not like this at all, let's dismiss simply the patch and 
save us the time
    
    i think #891 should definitely go in as there is not much to think about on 
that one. this one is fine, but the sorting should make sense for the full 
scope of identifiers from all different graphs. It can't be optimized to work 
for just TinkerGraph defaulted to `Integer` values - it wouldn't be fair to 
other graphs if we had that kind of favoritism :smile:  I can think about the 
sorting some more, but right now i'm not sure what would be best.


---

Reply via email to