-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55436/
-----------------------------------------------------------

(Updated Jan. 12, 2017, 7:21 p.m.)


Review request for atlas, David Kantor and Neeru Gupta.


Bugs: ATLAS-1386
    https://issues.apache.org/jira/browse/ATLAS-1386


Repository: atlas


Description
-------

When using the GraphBackedTypeStore, many unnecessary gremlin queries are 
executed as part of DSL query translated that are for things that cannot 
possibly be types, for example field names and aliases.  These changes 
eliminate unnecessary queries and consolidate the type lookup for trait and 
class types where it makes sense.


Diffs
-----

  repository/src/main/scala/org/apache/atlas/query/ClosureQuery.scala 
569d3f947e49f205b3d48c7387ee4df39938deff 
  repository/src/main/scala/org/apache/atlas/query/Expressions.scala 
ce21a597e79b684eaadc54f208f4a540d402f1b9 
  repository/src/main/scala/org/apache/atlas/query/QueryParser.scala 
42d76c84d48b1e13c3666be8de00fcc3f2bcdea9 
  repository/src/main/scala/org/apache/atlas/query/QueryProcessor.scala 
ee95a20a0bdd447d907b21a5d5a0642f4142bade 
  repository/src/main/scala/org/apache/atlas/query/Resolver.scala 
0957c57156381bfae82409a3c88083f564d790e8 
  repository/src/main/scala/org/apache/atlas/query/TypeUtils.scala 
dfa7093bec25e8954634af2a3f55f357eb7d46a1 
  
repository/src/test/java/org/apache/atlas/discovery/GraphBackedDiscoveryServiceTest.java
 1217e4af1c8bd63b191025ab9098b16ec0867a4c 
  repository/src/test/java/org/apache/atlas/query/QueryProcessorTest.java 
PRE-CREATION 
  typesystem/src/main/java/org/apache/atlas/typesystem/types/TypeSystem.java 
1358159d9cdc6f36f3a34d7946707d329b10f9d3 
  webapp/src/test/java/org/apache/atlas/web/resources/BaseResourceIT.java 
dcb126454b9efc0b50f6835e6dd1371f3e16d359 

Diff: https://reviews.apache.org/r/55436/diff/


Testing
-------

All Atlas tests run, no regressions found.  There were some tests that failed 
both with and without these changes.


Thanks,

Jeff Hagelberg

Reply via email to