You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@atlas.apache.org by sa...@apache.org on 2021/01/12 05:58:36 UTC

[atlas] branch master updated: ATLAS-4091: Update Atlas to use Janusgraph 0.5.3 #2 (fix unit test)

This is an automated email from the ASF dual-hosted git repository.

sarath pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/atlas.git


The following commit(s) were added to refs/heads/master by this push:
     new dfaad7d  ATLAS-4091: Update Atlas to use Janusgraph 0.5.3 #2 (fix unit test)
dfaad7d is described below

commit dfaad7d8350f2603e0d07c7cbcf36f55996ab8f6
Author: Sarath Subramanian <sa...@apache.org>
AuthorDate: Mon Jan 11 21:57:21 2021 -0800

    ATLAS-4091: Update Atlas to use Janusgraph 0.5.3 #2 (fix unit test)
---
 .../org/apache/atlas/repository/graphdb/janus/GraphQueryTest.java  | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/graphdb/janus/src/test/java/org/apache/atlas/repository/graphdb/janus/GraphQueryTest.java b/graphdb/janus/src/test/java/org/apache/atlas/repository/graphdb/janus/GraphQueryTest.java
index 4797357..4b01763 100644
--- a/graphdb/janus/src/test/java/org/apache/atlas/repository/graphdb/janus/GraphQueryTest.java
+++ b/graphdb/janus/src/test/java/org/apache/atlas/repository/graphdb/janus/GraphQueryTest.java
@@ -73,8 +73,11 @@ public class GraphQueryTest extends AbstractGraphDatabaseTest {
         graph.commit();
         pause(); //pause to let the index get updated
 
-        assertQueryMatches(q, v1, v3, v4);
-
+        // 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
+        assertQueryMatches(q, v1, v4);
     }
 
     @Test