-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/73157/
-----------------------------------------------------------

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

Reply via email to