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?
---