> On July 19, 2017, 10:54 a.m., David Radley wrote: > > graphdb/api/src/main/java/org/apache/atlas/repository/graphdb/AtlasGraph.java > > Line 147 (original), 147 (patched) > > <https://reviews.apache.org/r/60927/diff/1/?file=1778020#file1778020line147> > > > > I am unsure about the intended scope of this fix, I suggest that you > > introduce unit tests at this stage to test all the represtative queries > > that this fix will enable and error cases.
Will definitely add tests after I test out the usecases that it might solve. - Apoorv ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60927/#review180913 ----------------------------------------------------------- On July 17, 2017, 10:31 p.m., Apoorv Naik wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/60927/ > ----------------------------------------------------------- > > (Updated July 17, 2017, 10:31 p.m.) > > > Review request for atlas, Ashutosh Mestry, Madhan Neethiraj, Sarath > Subramanian, and Suma Shivaprasad. > > > Repository: atlas > > > Description > ------- > > Initial draft for multi-vertex graph query in atlas > > > Diffs > ----- > > > graphdb/api/src/main/java/org/apache/atlas/repository/graphdb/AtlasGraph.java > dded76f8 > > graphdb/api/src/main/java/org/apache/atlas/repository/graphdb/AtlasMultiVertexQuery.java > PRE-CREATION > > graphdb/common/src/main/java/org/apache/atlas/repository/graphdb/titan/query/NativeTitanMultiVertexQuery.java > PRE-CREATION > > graphdb/common/src/main/java/org/apache/atlas/repository/graphdb/titan/query/NativeTitanQueryFactory.java > ac7ff9e8 > graphdb/titan0/pom.xml ceda1bb2 > > graphdb/titan0/src/main/java/org/apache/atlas/repository/graphdb/titan0/Titan0Graph.java > 2408287b > > graphdb/titan0/src/main/java/org/apache/atlas/repository/graphdb/titan0/query/Titan0AtlasMultiVertexQuery.java > PRE-CREATION > > graphdb/titan1/src/main/java/org/apache/atlas/repository/graphdb/titan1/Titan1Graph.java > ffb6b37f > > graphdb/titan1/src/main/java/org/apache/atlas/repository/graphdb/titan1/query/Titan1AtlasMultiVertexQuery.java > PRE-CREATION > > repository/src/main/java/org/apache/atlas/discovery/ClassificationSearchProcessor.java > 77b2c7c2 > > > Diff: https://reviews.apache.org/r/60927/diff/1/ > > > Testing > ------- > > > Thanks, > > Apoorv Naik > >
