atlas-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ashutosh Mestry <>
Subject Review Request 65001: Atlas Advanced Search: Phase 3: Count, Sum, Max, Min with Group By Implementation
Date Sat, 06 Jan 2018 01:26:25 GMT

This is an automatically generated e-mail. To reply, visit:

Review request for atlas, Apoorv Naik, Madhan Neethiraj, and Sarath Subramanian.

Bugs: ATLAS-2229

Repository: atlas


This is the subsequent commit for the new Antlr-based DSL implementation.

This builds on top of what is already present.

The implementation of these operators does not impact the gremlin queries generated without
these operators. Alternatively stated, the approach applies the new implementation only when
on or any of these clauses are present within the DSL query.

Basic Gremlin is constructed. The 'select' clause is processed for one of these operations.
Additional closures are constructed and the filtered part of the query is passed to the closure.

**Bug fixes**
- Updates include pagination across results. This causes the query to fire again with the
appropriate values for _limit_ and _offset_ clauses.
- Fixed handling of date and boolean (true and false) values in _where_ clause.
- Error handling during parsing.
- Handling errors for invalid queries.

- Updated documentation, currently in draft. Separate patch will include updates to TWIKI.


  repository/src/main/java/org/apache/atlas/discovery/ cbe9a5b5

  repository/src/main/java/org/apache/atlas/query/ e4c81653 
  repository/src/main/java/org/apache/atlas/query/ a21e8843 
  repository/src/main/java/org/apache/atlas/query/ 6af1db67 
  repository/src/main/java/org/apache/atlas/query/ fe1c5bdc 
  repository/src/main/java/org/apache/atlas/query/ 5af8bc7b 
  repository/src/main/java/org/apache/atlas/query/antlr4/ 77c66c99 
  repository/src/main/java/org/apache/atlas/query/antlr4/AtlasDSLParser.g4 72d08e19 
  repository/src/main/java/org/apache/atlas/query/antlr4/ e557c4af 
  repository/src/main/java/org/apache/atlas/query/antlr4/ 4423c0e9

  repository/src/main/java/org/apache/atlas/query/antlr4/ 8c59d7a6

  repository/src/test/java/org/apache/atlas/query/ af4dc3e0 
  repository/src/test/java/org/apache/atlas/query/ ccf1cb29 
  repository/src/test/java/org/apache/atlas/services/ 8a06d949



**Unit test**
- Updated _GremlinComposerTest_ to include the new operations.
- Added 25+ new tests to _DSLQueriesTest_. See section _minMaxCountProvider_.

**Functional testing**
- Via web.
- Verified using hdfs_path samples.
- Verified using quick_start dataset.
- Verified using typical hive_db dataset (with 110 tables, each with 87 columns).

**REST API testing**
- V1 and V2 end points, via CURL.


Ashutosh Mestry

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message