-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56102/
-----------------------------------------------------------
(Updated Feb. 1, 2017, 9:41 p.m.)
Review request for atlas and David Kantor.
Changes
-------
Rebased changes, addressed code review comments -- now we're clearing the cache
explicitly at the end of each request.
Bugs: ATLAS-1388
https://issues.apache.org/jira/browse/ATLAS-1388
Repository: atlas
Description
-------
Cache created entities in RequestContext when they are created.
Update/refactor DefaultMetadataService.loadEntities() to use the cached
created entities.
Use the cache in the following places:
FullTextMapper
DefaultMetadataService.onEntitiesAdded - check cache before calling
DefaultMetadataService.loadEntities
DefaultMetadataService.onEntitiesUpdated - check cache before calling
DefaultMetadataService.loadEntities
EntityResource.getEntityDefinition - check cache before calling
DefaultMetadataService.getEntityDefinition
EntityResource.getResponse - check cache before calling
DefaultMetadataService.getEntityDefinition
Diffs (updated)
-----
repository/src/main/java/org/apache/atlas/discovery/graph/DefaultGraphPersistenceStrategy.java
f4d8f00da863deeb98209aec2dfc4ccb22734cfa
repository/src/main/java/org/apache/atlas/repository/graph/FullTextMapper.java
5be8d0bc349294dc4340ffe5831bfb1265825ed8
repository/src/main/java/org/apache/atlas/repository/graph/GraphBackedMetadataRepository.java
6608551be44033f07ddbd2ac3f3764d1ac3b0f22
repository/src/main/java/org/apache/atlas/repository/graph/GraphToTypedInstanceMapper.java
38a553abf0151221c5d802e9ce1977a693d1a698
repository/src/main/java/org/apache/atlas/repository/graph/TypedInstanceToGraphMapper.java
bae8b2ac0cae6bf6392f91f24030d81089ae660b
repository/src/test/java/org/apache/atlas/TestUtils.java
cda9eac38861fd55c1494d6d90b5e750e6133545
repository/src/test/java/org/apache/atlas/discovery/GraphBackedDiscoveryServiceTest.java
d447c2d90fec6c03c7fecb8d8ef57a6ea072d35a
repository/src/test/java/org/apache/atlas/repository/graph/GraphBackedMetadataRepositoryDeleteTestBase.java
9e850a9dc0ac122cf3fcdbf7df72a93643abc98d
repository/src/test/java/org/apache/atlas/repository/graph/GraphBackedMetadataRepositoryTest.java
7444bf38834f59454d67cf763c932d665d7ef31c
repository/src/test/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityStoreV1Test.java
c902f8126f6b516e84ab9d89b0ec80ac8ed819fd
repository/src/test/java/org/apache/atlas/service/DefaultMetadataServiceTest.java
03ef4fe02f9b4eba273297ea954f00c9181f1f02
server-api/src/main/java/org/apache/atlas/RequestContext.java
651a71dc238a3adaac9504a77ba439785fa07ca8
Diff: https://reviews.apache.org/r/56102/diff/
Testing
-------
Ran full build, no regresssions found.
Thanks,
Jeff Hagelberg
