Github user twilmes commented on a diff in the pull request:

    https://github.com/apache/tinkerpop/pull/891#discussion_r204065136
  
    --- Diff: 
gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/graphml/GraphMLWriter.java
 ---
    @@ -218,20 +220,33 @@ private void writeTypes(final Map<String, String> 
identifiedVertexKeyTypes,
                                 final Map<String, String> 
identifiedEdgeKeyTypes,
                                 final XMLStreamWriter writer) throws 
XMLStreamException {
             // <key id="weight" for="edge" attr.name="weight" 
attr.type="float"/>
    -        final Collection<String> vertexKeySet = 
getVertexKeysAndNormalizeIfRequired(identifiedVertexKeyTypes);
    +        Collection<String> vertexKeySet = 
getVertexKeysAndNormalizeIfRequired(identifiedVertexKeyTypes);
    --- End diff --
    
    Could these still be made `final`?


---

Reply via email to