Ok, thanks!
From: Madhan Neethiraj [mailto:[email protected]] Sent: Monday, January 30, 2017 10:55 PM To: Jeffrey N Hagelberg <[email protected]>; Fnu Neerju <[email protected]>; Dave Kantor <[email protected]> Cc: Sarath Subramanian <[email protected]>; Madhan Neethiraj <[email protected]>; atlas <[email protected]>; Vimal Sharma <[email protected]> Subject: Re: Review Request 55443: ATLAS-1387 : Compiled Query Cache Jeff – I am about to commit the fix for unit test failure (introduced in my earlier commit); I will update CompiledQueryCacheKeyTest.java with the license header text. Sarath – thanks for investigating at the unit test failure and the fix! Thanks, Madhan From: Jeff Hagelberg <[email protected] <mailto:[email protected]> > on behalf of Jeff Hagelberg <[email protected] <mailto:[email protected]> > Reply-To: Jeff Hagelberg <[email protected] <mailto:[email protected]> > Date: Monday, January 30, 2017 at 2:29 PM To: Neeru Gupta <[email protected] <mailto:[email protected]> >, David Kantor <[email protected] <mailto:[email protected]> > Cc: Sarath Subramanian <[email protected] <mailto:[email protected]> >, Jeff Hagelberg <[email protected] <mailto:[email protected]> >, Madhan Neethiraj <[email protected] <mailto:[email protected]> >, atlas <[email protected] <mailto:[email protected]> >, Vimal Sharma <[email protected] <mailto:[email protected]> > Subject: Re: Review Request 55443: ATLAS-1387 : Compiled Query Cache This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55443/ On January 30th, 2017, 7:03 p.m. UTC, Sarath Subramanian wrote: <https://reviews.apache.org/r/55443/diff/10/?file=1618378#file1618378line1> repository/src/test/java/org/apache/atlas/util/CompiledQueryCacheKeyTest.java (Diff revision 10) 1 package org.apache.atlas.util; Could you add Apache license header here to avoid rat failures. Unfortunately, this was already committed. I'm working on putting together a patch for ATLAS-1388. I'll fix the RAT issue in there. I'm expecting to be submitting that for review very soon. - Jeff On January 29th, 2017, 12:18 a.m. UTC, Jeff Hagelberg wrote: Review request for atlas, David Kantor and Neeru Gupta. By Jeff Hagelberg. Updated Jan. 29, 2017, 12:18 a.m. Bugs: ATLAS-1387 <https://issues.apache.org/jira/browse/ATLAS-1387> Repository: atlas Description Adds a compiled query cache to Atlas. This avoids the overhead of parsing and translating DSL queries that have been previously executed. Testing Ran all atlas tests, no regressions found. There were some tests that failed both with and without these changes. Diffs * common/pom.xml (0226541b2dd73e8a01f050982cb8b37f5fed9cab) * common/src/main/java/org/apache/atlas/utils/LruCache.java (PRE-CREATION) * common/src/test/java/org/apache/atlas/utils/LruCacheTest.java (PRE-CREATION) * distro/src/conf/atlas-application.properties (303ce7b00c93f2499e7b18c5f3a131b50c150b69) * repository/src/main/java/org/apache/atlas/discovery/graph/GraphBackedDiscoveryService.java (fb488cdfc7cef195ffc5221afb9a9109a6e92fc9) * repository/src/main/java/org/apache/atlas/util/AtlasRepositoryConfiguration.java (6655085ee11c94addca1564cf77bbdb001c4586f) * repository/src/main/java/org/apache/atlas/util/CompiledQueryCacheKey.java (PRE-CREATION) * repository/src/main/java/org/apache/atlas/util/NoopGremlinQuery.java (PRE-CREATION) * repository/src/main/scala/org/apache/atlas/query/QueryProcessor.scala (5693c9ecc1a96154a8b0ac48256e5074dc041c5f) * repository/src/test/java/org/apache/atlas/util/CompiledQueryCacheKeyTest.java (PRE-CREATION) View Diff <https://reviews.apache.org/r/55443/diff/>
