----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/73157/#review222526 -----------------------------------------------------------
repository/src/test/java/org/apache/atlas/query/GremlinQueryComposerTest.java Lines 389 (patched) <https://reviews.apache.org/r/73157/#comment311630> In the _DSLQueries_ test, please add a test that includes multiple AND/OR clauses. - Ashutosh Mestry On Jan. 21, 2021, 11:26 p.m., Deep Singh wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/73157/ > ----------------------------------------------------------- > > (Updated Jan. 21, 2021, 11:26 p.m.) > > > Review request for atlas, Ashutosh Mestry, Madhan Neethiraj, Pinal Shah, and > Sarath Subramanian. > > > Bugs: ATLAS-4110 > https://issues.apache.org/jira/browse/ATLAS-4110 > > > Repository: atlas > > > Description > ------- > > After upgrading Janusgraph to V-0.5.3, behaviour of DSL queries with neq in > where-clause changed. > > /* > Janusgraph 0.5.3 introduced changes to NOT-EQUAL operator > https://github.com/JanusGraph/janusgraph/issues/2205 > Earlier behavior doesn't check if property exists, so all vertices without > this property was also returned in 'neq'. > Now 'neq' checks if property exists and property is not-equal to property > value > */ > > To handle this case I have introduced HAS_NOT_OPERATOR which basically is an > OR operation of HAS_OPERATOR(with neq check) and HAS_NOT_PROPERTY(checks > absence of a property). > > > Diffs > ----- > > repository/src/main/java/org/apache/atlas/query/GremlinClause.java > 7a98dddbe > repository/src/main/java/org/apache/atlas/query/GremlinQueryComposer.java > c5c53fc7f > > repository/src/test/java/org/apache/atlas/query/GremlinQueryComposerTest.java > 96e284069 > > > Diff: https://reviews.apache.org/r/73157/diff/1/ > > > Testing > ------- > > Manual testing was done. > Unit test is added. > > > Thanks, > > Deep Singh > >
