fix(interactive): fix bugs when try to get labels or properties from an None entry

interactive_engine/compiler/src/main/java/com/alibaba/graphscope/cypher/integration/suite/simple/SimpleMatchQueries.java

@@ -219,4 +219,15 @@ public static QueryContext get_simple_match_query_17_test() {
         List<String> expected = Arrays.asList("Record<{$f0: 851}>");
         return new QueryContext(query, expected);
     }
+
+    public static QueryContext get_simple_match_query_18_test() {
+        String query =
+                "MATCH (country:PLACE {name:"
+                        + " \"India\"})<-[:ISPARTOF]-(:PLACE)<-[:ISLOCATEDIN]-(zombie:PERSON)\n"
+                        + "OPTIONAL MATCH (zombie)<-[:HASCREATOR]-(message)\n"
+                        + "WHERE message.creationDate < 20100630000000000\n"
+                        + " Return count(country);";
+        List<String> expected = Arrays.asList("Record<{$f0: 25}>");
+        return new QueryContext(query, expected);
+    }
 }

interactive_engine/compiler/src/test/java/com/alibaba/graphscope/cypher/integration/ldbc/SimpleMatchTest.java

@@ -162,6 +162,13 @@ public void run_simple_match_17_test() {
         Assert.assertEquals(testQuery.getExpectedResult().toString(), result.list().toString());
     }
 
+    @Test
+    public void run_simple_match_18_test() {
+        QueryContext testQuery = SimpleMatchQueries.get_simple_match_query_18_test();
+        Result result = session.run(testQuery.getQuery());
+        Assert.assertEquals(testQuery.getExpectedResult().toString(), result.list().toString());
+    }
+
     @AfterClass
     public static void afterClass() {
         if (session != null) {

interactive_engine/executor/ir/graph_proxy/src/apis/graph/element/property.rs

@@ -44,40 +44,44 @@ impl PropKey {
         let prop_obj = if let PropKey::Len = self {
             element.len().into()
         } else {
-            let graph_element = element
-                .as_graph_element()
-                .ok_or_else(|| ExprEvalError::UnexpectedDataType(self.into()))?;
-            match self {
-                PropKey::Id => graph_element.id().into(),
-                PropKey::Label => graph_element
-                    .label()
-                    .map(|label| label.into())
-                    .unwrap_or(Object::None),
-                PropKey::Len => unreachable!(),
-                PropKey::All => graph_element
-                    .get_all_properties()
-                    .map(|obj| {
-                        obj.into_iter()
-                            .map(|(key, value)| {
-                                let obj_key: Object = match key {
-                                    NameOrId::Str(str) => str.into(),
-                                    NameOrId::Id(id) => id.into(),
-                                };
-                                (obj_key, value)
-                            })
-                            .collect::<Vec<(Object, Object)>>()
-                            .into()
-                    })
-                    .unwrap_or(Object::None),
-                PropKey::Key(key) => {
-                    if let Some(prop_val) = graph_element.get_property(key) {
-                        prop_val.try_to_owned().ok_or_else(|| {
-                            ExprEvalError::OtherErr("cannot get `Object` from `BorrowObject`".to_string())
-                        })?
-                    } else {
-                        Object::None
+            if let Some(graph_element) = element.as_graph_element() {
+                match self {
+                    PropKey::Id => graph_element.id().into(),
+                    PropKey::Label => graph_element
+                        .label()
+                        .map(|label| label.into())
+                        .unwrap_or(Object::None),
+                    PropKey::Len => unreachable!(),
+                    PropKey::All => graph_element
+                        .get_all_properties()
+                        .map(|obj| {
+                            obj.into_iter()
+                                .map(|(key, value)| {
+                                    let obj_key: Object = match key {
+                                        NameOrId::Str(str) => str.into(),
+                                        NameOrId::Id(id) => id.into(),
+                                    };
+                                    (obj_key, value)
+                                })
+                                .collect::<Vec<(Object, Object)>>()
+                                .into()
+                        })
+                        .unwrap_or(Object::None),
+                    PropKey::Key(key) => {
+                        if let Some(prop_val) = graph_element.get_property(key) {
+                            prop_val.try_to_owned().ok_or_else(|| {
+                                ExprEvalError::OtherErr(
+                                    "cannot get `Object` from `BorrowObject`".to_string(),
+                                )
+                            })?
+                        } else {
+                            Object::None
+                        }
                     }
                 }
+            } else {
+                // if try to get property from non-graph-element, return None
+                Object::None
             }
         };
         Ok(prop_obj)

interactive_engine/executor/ir/runtime/src/process/operator/map/get_v.rs

@@ -158,7 +158,8 @@ impl FilterMapFunction<Record, Record> for AuxiliaOperator {
             //     then we set tag=None and alias="a" in auxilia
 
             // 1. If to filter by labels, and the entry itself carries label information already, directly eval it without query the store
-            if self.query_params.has_labels() && entry.label().is_some() {
+            if self.query_params.has_labels() && entry.as_graph_path().is_some() && entry.label().is_some()
+            {
                 if !self
                     .query_params
                     .labels