----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64800/ -----------------------------------------------------------
(Updated Dec. 23, 2017, 12:13 a.m.) Review request for atlas and Madhan Neethiraj. Changes ------- Rebased on latest master Bugs: ATLAS-2229 https://issues.apache.org/jira/browse/ATLAS-2229 Repository: atlas Description ------- Gremlin related changes -------------------------------------------------------------------- Global aggregator instead of local, no implicit limits on aggregator Error handling for malformed queries Simplified outgoing traversal Fix for groupBy followed by orderBy Handled quoted string labels when using aggregator functions -------------------------------------------------------------------- Removed space/comma separated query from DSL grammar Diffs (updated) ----- intg/src/main/java/org/apache/atlas/AtlasErrorCode.java fb741f8f6 repository/src/main/java/org/apache/atlas/query/AtlasDSL.java PRE-CREATION repository/src/main/java/org/apache/atlas/query/DSLVisitor.java b597a0daa repository/src/main/java/org/apache/atlas/query/Expressions.java 77350f09b repository/src/main/java/org/apache/atlas/query/GremlinTranslator.java 6ce90e620 repository/src/main/java/org/apache/atlas/query/IdentifierHelper.java 0c7e2ed83 repository/src/main/java/org/apache/atlas/query/Lookup.java a64b68876 repository/src/main/java/org/apache/atlas/query/QueryParser.java 6e378a1f6 repository/src/main/java/org/apache/atlas/query/QueryProcessor.java 5e07a9228 repository/src/main/java/org/apache/atlas/query/antlr4/AtlasDSLLexer.java 576e1298e repository/src/main/java/org/apache/atlas/query/antlr4/AtlasDSLLexer.tokens 8c147b4fc repository/src/main/java/org/apache/atlas/query/antlr4/AtlasDSLParser.g4 058a5c80f repository/src/main/java/org/apache/atlas/query/antlr4/AtlasDSLParser.java e557c4af2 repository/src/main/java/org/apache/atlas/query/antlr4/AtlasDSLParser.tokens 8c147b4fc repository/src/main/java/org/apache/atlas/query/antlr4/AtlasDSLParserBaseVisitor.java 4423c0e92 repository/src/main/java/org/apache/atlas/query/antlr4/AtlasDSLParserVisitor.java 8c59d7a6e repository/src/test/java/org/apache/atlas/query/QueryProcessorTest.java b87314199 Diff: https://reviews.apache.org/r/64800/diff/2/ Changes: https://reviews.apache.org/r/64800/diff/1-2/ Testing ------- Testing done via UI UTs and ITs have not been tackled yet, once all the existing cases are implemented that's the next big thing. Thanks, Apoorv Naik