-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64800/
-----------------------------------------------------------
Review request for atlas and Madhan Neethiraj.
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
-----
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 76a10e3d5
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/1/
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
