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

    https://github.com/apache/metamodel/pull/179#discussion_r189893280
  
    --- Diff: 
neo4j/src/main/java/org/apache/metamodel/neo4j/Neo4jDataContext.java ---
    @@ -236,8 +256,8 @@ protected String getMainSchemaName() throws 
MetaModelException {
             }
         }
     
    -    private List<JSONObject> getOutgoingRelationshipsPerNode(Integer 
nodeId) {
    -        List<JSONObject> outgoingRelationshipsPerNode = new 
ArrayList<JSONObject>();
    +    private List<JSONObject> getOutgoingRelationshipsPerNode(final Integer 
nodeId) {
    +        List<JSONObject> outgoingRelationshipsPerNode = new ArrayList<>();
    --- End diff --
    
    If you improve this line of code, why not make the variable `final` too?


---

Reply via email to