----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62455/#review185835 -----------------------------------------------------------
graphdb/api/src/main/java/org/apache/atlas/repository/graphdb/AtlasGraphManagement.java Lines 69 (patched) <https://reviews.apache.org/r/62455/#comment262188> The test to validate this GraphBackedMetadataRepositoryTest.testConcurrentCalls() is tweaked to work because of this issue. This needs to change => line#180 assertTrue(validated1 && validated2); - Sarath Subramanian On Sept. 20, 2017, 2:55 p.m., Apoorv Naik wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/62455/ > ----------------------------------------------------------- > > (Updated Sept. 20, 2017, 2:55 p.m.) > > > Review request for atlas and Madhan Neethiraj. > > > Bugs: ATLAS-2092 > https://issues.apache.org/jira/browse/ATLAS-2092 > > > Repository: atlas > > > Description > ------- > > See ATLAS-2092 for the discussion > > > Diffs > ----- > > > graphdb/api/src/main/java/org/apache/atlas/repository/graphdb/AtlasGraphManagement.java > 0db46d46 > > graphdb/titan0/src/main/java/org/apache/atlas/repository/graphdb/titan0/Titan0GraphManagement.java > ec4d6c45 > > graphdb/titan1/src/main/java/org/apache/atlas/repository/graphdb/titan1/Titan1GraphManagement.java > 61dc2980 > > repository/src/main/java/org/apache/atlas/repository/graph/GraphBackedSearchIndexer.java > be6a0a06 > > > Diff: https://reviews.apache.org/r/62455/diff/1/ > > > Testing > ------- > > mvn clean package -Dberkeley-elasticsearch went through fine > Build was done 3-4 times to ensure that the concurrency tests passed every > time > > > Thanks, > > Apoorv Naik > >