diff --git a/src/clients/meta/MetaClient.cpp b/src/clients/meta/MetaClient.cpp index 8e095331603..ddaa64fd897 100644 --- a/src/clients/meta/MetaClient.cpp +++ b/src/clients/meta/MetaClient.cpp @@ -829,13 +829,13 @@ Status MetaClient::handleResponse(const RESP& resp) { case nebula::cpp2::ErrorCode::E_EXISTED: return Status::Error("Existed!"); case nebula::cpp2::ErrorCode::E_SPACE_NOT_FOUND: - return Status::Error("Space not existed!"); + return Status::SpaceNotFound("Space not existed!"); case nebula::cpp2::ErrorCode::E_TAG_NOT_FOUND: - return Status::Error("Tag not existed!"); + return Status::TagNotFound("Tag not existed!"); case nebula::cpp2::ErrorCode::E_EDGE_NOT_FOUND: - return Status::Error("Edge not existed!"); + return Status::EdgeNotFound("Edge not existed!"); case nebula::cpp2::ErrorCode::E_INDEX_NOT_FOUND: - return Status::Error("Index not existed!"); + return Status::IndexNotFound("Index not existed!"); case nebula::cpp2::ErrorCode::E_STATS_NOT_FOUND: return Status::Error( "There is no any stats info to show, please execute " @@ -1389,7 +1389,7 @@ StatusOr MetaClient::getSpaceIdByNameFromCache(const std::string& if (it != metadata.spaceIndexByName_.end()) { return it->second; } - return Status::SpaceNotFound(); + return Status::SpaceNotFound(fmt::format("SpaceName `{}`", name)); } StatusOr MetaClient::getSpaceNameByIdFromCache(GraphSpaceID spaceId) { @@ -1401,7 +1401,7 @@ StatusOr MetaClient::getSpaceNameByIdFromCache(GraphSpaceID spaceId auto spaceIt = metadata.localCache_.find(spaceId); if (spaceIt == metadata.localCache_.end()) { LOG(ERROR) << "Space " << spaceId << " not found!"; - return Status::Error("Space %d not found", spaceId); + return Status::SpaceNotFound(fmt::format("SpaceId `{}`", spaceId)); } return spaceIt->second->spaceDesc_.get_space_name(); } @@ -1415,7 +1415,7 @@ StatusOr MetaClient::getTagIDByNameFromCache(const GraphSpaceID& space, const auto& metadata = *metadata_.load(); auto it = metadata.spaceTagIndexByName_.find(std::make_pair(space, name)); if (it == metadata.spaceTagIndexByName_.end()) { - return Status::Error("TagName `%s' is nonexistent", name.c_str()); + return Status::TagNotFound(fmt::format("TagName `{}`", name)); } return it->second; } @@ -1429,7 +1429,7 @@ StatusOr MetaClient::getTagNameByIdFromCache(const GraphSpaceID& sp const auto& metadata = *metadata_.load(); auto it = metadata.spaceTagIndexById_.find(std::make_pair(space, tagId)); if (it == metadata.spaceTagIndexById_.end()) { - return Status::Error("TagID `%d' is nonexistent", tagId); + return Status::TagNotFound(fmt::format("TagID `{}`", tagId)); } return it->second; } @@ -1443,7 +1443,7 @@ StatusOr MetaClient::getEdgeTypeByNameFromCache(const GraphSpaceID& sp const auto& metadata = *metadata_.load(); auto it = metadata.spaceEdgeIndexByName_.find(std::make_pair(space, name)); if (it == metadata.spaceEdgeIndexByName_.end()) { - return Status::Error("EdgeName `%s' is nonexistent", name.c_str()); + return Status::EdgeNotFound(fmt::format("EdgeName `{}`", name)); } return it->second; } @@ -1457,7 +1457,7 @@ StatusOr MetaClient::getEdgeNameByTypeFromCache(const GraphSpaceID& const auto& metadata = *metadata_.load(); auto it = metadata.spaceEdgeIndexByType_.find(std::make_pair(space, edgeType)); if (it == metadata.spaceEdgeIndexByType_.end()) { - return Status::Error("EdgeType `%d' is nonexistent", edgeType); + return Status::EdgeNotFound(fmt::format("EdgeType `{}`", edgeType)); } return it->second; } @@ -1470,7 +1470,7 @@ StatusOr> MetaClient::getAllEdgeFromCache(const GraphSp const auto& metadata = *metadata_.load(); auto it = metadata.spaceAllEdgeMap_.find(space); if (it == metadata.spaceAllEdgeMap_.end()) { - return Status::Error("SpaceId `%d' is nonexistent", space); + return Status::SpaceNotFound(fmt::format("SpaceId `{}`", space)); } return it->second; } @@ -1542,7 +1542,7 @@ StatusOr MetaClient::partsNum(GraphSpaceID spaceId) { const auto& metadata = *metadata_.load(); auto it = metadata.localCache_.find(spaceId); if (it == metadata.localCache_.end()) { - return Status::Error("Space not found, spaceid: %d", spaceId); + return Status::SpaceNotFound(fmt::format("SpaceId `{}`", spaceId)); } return it->second->partsAlloc_.size(); } @@ -1937,7 +1937,7 @@ StatusOr MetaClient::getSpaceVidLen(const GraphSpaceID& spaceId) { auto spaceIt = metadata.localCache_.find(spaceId); if (spaceIt == metadata.localCache_.end()) { LOG(ERROR) << "Space " << spaceId << " not found!"; - return Status::Error("Space %d not found", spaceId); + return Status::SpaceNotFound(fmt::format("SpaceId `{}`", spaceId)); } auto& vidType = spaceIt->second->spaceDesc_.get_vid_type(); auto vIdLen = vidType.type_length_ref().has_value() ? *vidType.get_type_length() : 0; @@ -1956,7 +1956,7 @@ StatusOr MetaClient::getSpaceVidType(const GraphSpac auto spaceIt = metadata.localCache_.find(spaceId); if (spaceIt == metadata.localCache_.end()) { LOG(ERROR) << "Space " << spaceId << " not found!"; - return Status::Error("Space %d not found", spaceId); + return Status::SpaceNotFound(fmt::format("SpaceId `{}`", spaceId)); } auto vIdType = spaceIt->second->spaceDesc_.get_vid_type().get_type(); if (vIdType != nebula::cpp2::PropertyType::INT64 && @@ -1969,16 +1969,16 @@ StatusOr MetaClient::getSpaceVidType(const GraphSpac return vIdType; } -StatusOr MetaClient::getSpaceDesc(const GraphSpaceID& space) { +StatusOr MetaClient::getSpaceDesc(const GraphSpaceID& spaceId) { if (!ready_) { return Status::Error("Not ready!"); } folly::rcu_reader guard; const auto& metadata = *metadata_.load(); - auto spaceIt = metadata.localCache_.find(space); + auto spaceIt = metadata.localCache_.find(spaceId); if (spaceIt == metadata.localCache_.end()) { - LOG(ERROR) << "Space " << space << " not found!"; - return Status::Error("Space %d not found", space); + LOG(ERROR) << "Space " << spaceId << " not found!"; + return Status::SpaceNotFound(fmt::format("SpaceId `{}`", spaceId)); } return spaceIt->second->spaceDesc_; } @@ -2042,7 +2042,7 @@ StatusOr MetaClient::getAllVerTagSchema(GraphSpaceID spaceId) { const auto& metadata = *metadata_.load(); auto iter = metadata.localCache_.find(spaceId); if (iter == metadata.localCache_.end()) { - return Status::Error("Space %d not found", spaceId); + return Status::SpaceNotFound(fmt::format("SpaceId `{}`", spaceId)); } return iter->second->tagSchemas_; } @@ -2055,7 +2055,7 @@ StatusOr MetaClient::getAllLatestVerTagSchema(const GraphSpaceID& spa const auto& metadata = *metadata_.load(); auto iter = metadata.localCache_.find(spaceId); if (iter == metadata.localCache_.end()) { - return Status::Error("Space %d not found", spaceId); + return Status::SpaceNotFound(fmt::format("SpaceId `{}`", spaceId)); } TagSchema tagsSchema; tagsSchema.reserve(iter->second->tagSchemas_.size()); @@ -2074,7 +2074,7 @@ StatusOr MetaClient::getAllVerEdgeSchema(GraphSpaceID spaceId) { const auto& metadata = *metadata_.load(); auto iter = metadata.localCache_.find(spaceId); if (iter == metadata.localCache_.end()) { - return Status::Error("Space %d not found", spaceId); + return Status::SpaceNotFound(fmt::format("SpaceId `{}`", spaceId)); } return iter->second->edgeSchemas_; } @@ -2087,7 +2087,7 @@ StatusOr MetaClient::getAllLatestVerEdgeSchemaFromCache(const GraphS const auto& metadata = *metadata_.load(); auto iter = metadata.localCache_.find(spaceId); if (iter == metadata.localCache_.end()) { - return Status::Error("Space %d not found", spaceId); + return Status::SpaceNotFound(fmt::format("SpaceId `{}`", spaceId)); } EdgeSchema edgesSchema; edgesSchema.reserve(iter->second->edgeSchemas_.size()); @@ -2140,7 +2140,7 @@ StatusOr> MetaClient::getTagIndexByNameFromCach std::pair key(space, name); auto iter = tagNameIndexMap_.find(key); if (iter == tagNameIndexMap_.end()) { - return Status::IndexNotFound(); + return Status::IndexNotFound(fmt::format("Index: {}:{}", space, name)); } auto indexID = iter->second; auto itemStatus = getTagIndexFromCache(space, indexID); @@ -2158,7 +2158,7 @@ StatusOr> MetaClient::getEdgeIndexByNameFromCac std::pair key(space, name); auto iter = edgeNameIndexMap_.find(key); if (iter == edgeNameIndexMap_.end()) { - return Status::IndexNotFound(); + return Status::IndexNotFound(fmt::format("Index: {}:{}", space, name)); } auto indexID = iter->second; auto itemStatus = getEdgeIndexFromCache(space, indexID); @@ -2179,7 +2179,7 @@ StatusOr> MetaClient::getTagIndexFromCache(Grap auto spaceIt = metadata.localCache_.find(spaceId); if (spaceIt == metadata.localCache_.end()) { VLOG(3) << "Space " << spaceId << " not found!"; - return Status::SpaceNotFound(); + return Status::SpaceNotFound(fmt::format("SpaceId `{}`", spaceId)); } else { auto iter = spaceIt->second->tagIndexes_.find(indexID); if (iter == spaceIt->second->tagIndexes_.end()) { @@ -2207,7 +2207,7 @@ StatusOr MetaClient::getRelatedTagIDByIndexNameFromCache(const GraphSpace } StatusOr> MetaClient::getEdgeIndexFromCache(GraphSpaceID spaceId, - IndexID indexID) { + IndexID indexId) { if (!ready_) { return Status::Error("Not ready!"); } @@ -2217,12 +2217,12 @@ StatusOr> MetaClient::getEdgeIndexFromCache(Gra auto spaceIt = metadata.localCache_.find(spaceId); if (spaceIt == metadata.localCache_.end()) { VLOG(3) << "Space " << spaceId << " not found!"; - return Status::SpaceNotFound(); + return Status::SpaceNotFound(fmt::format("SpaceId `{}`", spaceId)); } else { - auto iter = spaceIt->second->edgeIndexes_.find(indexID); + auto iter = spaceIt->second->edgeIndexes_.find(indexId); if (iter == spaceIt->second->edgeIndexes_.end()) { - VLOG(3) << "Space " << spaceId << ", Edge Index " << indexID << " not found!"; - return Status::IndexNotFound(); + VLOG(3) << "Space " << spaceId << ", Edge Index " << indexId << " not found!"; + return Status::IndexNotFound(fmt::format("Index: {}:{}", spaceId, indexId)); } else { return iter->second; } @@ -2255,7 +2255,7 @@ StatusOr>> MetaClient::getTagIndexe auto spaceIt = metadata.localCache_.find(spaceId); if (spaceIt == metadata.localCache_.end()) { VLOG(3) << "Space " << spaceId << " not found!"; - return Status::SpaceNotFound(); + return Status::SpaceNotFound(fmt::format("SpaceId `{}`", spaceId)); } else { auto tagIndexes = spaceIt->second->tagIndexes_; auto iter = tagIndexes.begin(); @@ -2279,7 +2279,7 @@ StatusOr>> MetaClient::getEdgeIndex auto spaceIt = metadata.localCache_.find(spaceId); if (spaceIt == metadata.localCache_.end()) { VLOG(3) << "Space " << spaceId << " not found!"; - return Status::SpaceNotFound(); + return Status::SpaceNotFound(fmt::format("SpaceId `{}`", spaceId)); } else { auto edgeIndexes = spaceIt->second->edgeIndexes_; auto iter = edgeIndexes.begin(); @@ -2490,7 +2490,7 @@ StatusOr MetaClient::getLatestEdgeVersionFromCache(const GraphSpaceID const auto& metadata = *metadata_.load(); auto it = metadata.spaceNewestEdgeVerMap_.find(std::make_pair(space, edgeType)); if (it == metadata.spaceNewestEdgeVerMap_.end()) { - return Status::EdgeNotFound(); + return Status::EdgeNotFound(fmt::format("EdgeType `{}`", edgeType)); } return it->second; } diff --git a/src/graph/validator/test/MockSchemaManager.cpp b/src/graph/validator/test/MockSchemaManager.cpp index 6236d73212e..3c5a88c50c3 100644 --- a/src/graph/validator/test/MockSchemaManager.cpp +++ b/src/graph/validator/test/MockSchemaManager.cpp @@ -99,7 +99,7 @@ StatusOr MockSchemaManager::toGraphSpaceID(folly::StringPiece spac if (findIt != spaceNameIds_.end()) { return findIt->second; } - return Status::Error("Space `%s' not found", spaceName.str().c_str()); + return Status::SpaceNotFound("Space `%s' not found", spaceName.str().c_str()); } StatusOr MockSchemaManager::toGraphSpaceName(GraphSpaceID space) { @@ -108,7 +108,7 @@ StatusOr MockSchemaManager::toGraphSpaceName(GraphSpaceID space) { return s.first; } } - return Status::Error("Space `%d' not found", space); + return Status::SpaceNotFound("Space `%d' not found", space); } StatusOr MockSchemaManager::toTagID(GraphSpaceID space, const folly::StringPiece tagName) { @@ -120,7 +120,7 @@ StatusOr MockSchemaManager::toTagID(GraphSpaceID space, const folly::Stri if (tagFindIt != tagNameIds_.end()) { return tagFindIt->second; } - return Status::Error("TagName `%s' not found", tagName.str().c_str()); + return Status::TagNotFound("TagName `%s' not found", tagName.str().c_str()); } StatusOr MockSchemaManager::toTagName(GraphSpaceID space, TagID tagId) { @@ -132,7 +132,7 @@ StatusOr MockSchemaManager::toTagName(GraphSpaceID space, TagID tag if (tagFindIt != tagIdNames_.end()) { return tagFindIt->second; } - return Status::Error("TagID `%d' not found", tagId); + return Status::TagNotFound("TagID `%d' not found", tagId); } StatusOr MockSchemaManager::toEdgeType(GraphSpaceID space, folly::StringPiece typeName) { @@ -144,7 +144,7 @@ StatusOr MockSchemaManager::toEdgeType(GraphSpaceID space, folly::Stri if (edgeFindIt != edgeNameIds_.end()) { return edgeFindIt->second; } - return Status::Error("EdgeName `%s' not found", typeName.str().c_str()); + return Status::EdgeNotFound("EdgeName `%s' not found", typeName.str().c_str()); } StatusOr MockSchemaManager::toEdgeName(GraphSpaceID space, EdgeType edgeType) { @@ -156,7 +156,7 @@ StatusOr MockSchemaManager::toEdgeName(GraphSpaceID space, EdgeType if (edgeFindIt != edgeIdNames_.end()) { return edgeFindIt->second; } - return Status::Error("EdgeType `%d' not found", edgeType); + return Status::EdgeNotFound("EdgeType `%d' not found", edgeType); } StatusOr> MockSchemaManager::getAllEdge(GraphSpaceID) { diff --git a/src/graph/visitor/PrunePropertiesVisitor.cpp b/src/graph/visitor/PrunePropertiesVisitor.cpp index 2bbefa74c1e..263171c1719 100644 --- a/src/graph/visitor/PrunePropertiesVisitor.cpp +++ b/src/graph/visitor/PrunePropertiesVisitor.cpp @@ -51,6 +51,13 @@ void PrunePropertiesVisitor::visitCurrent(Project *node) { auto *expr = col->expr(); status_ = extractPropsFromExpr(expr); if (!status_.ok()) { + // Project a not exit tag, should not break other columns + // e.g. Vertex tage {{"name", "string"}, {"age", "int64"}} + // Project "... RETURN v.name, v.xxx.yyy, v.player.age" + // v.xxx.yyy should not break v.player.age + if (status_.isTagNotFound()) { + continue; + } return; } } diff --git a/src/meta/test/MetaClientTest.cpp b/src/meta/test/MetaClientTest.cpp index f79c47b9df7..affb25e7a26 100644 --- a/src/meta/test/MetaClientTest.cpp +++ b/src/meta/test/MetaClientTest.cpp @@ -725,7 +725,7 @@ TEST(MetaClientTest, TagIndexTest) { client->createTagIndex(space, "tag_not_exist_index", "tag_not_exist", std::move(fields)) .get(); ASSERT_FALSE(result.ok()); - ASSERT_EQ(Status::Error("not existed!"), result.status()); + ASSERT_EQ(Status::TagNotFound("not existed!"), result.status()); } { std::vector fields; diff --git a/tests/tck/features/mutate/ClearSpace.feature b/tests/tck/features/mutate/ClearSpace.feature index 5942c7ded04..e24ae3a1513 100644 --- a/tests/tck/features/mutate/ClearSpace.feature +++ b/tests/tck/features/mutate/ClearSpace.feature @@ -31,7 +31,7 @@ Feature: Clear space test """ CLEAR SPACE clear_space_0; """ - Then a ExecutionError should be raised at runtime: Space not existed! + Then a ExecutionError should be raised at runtime: SpaceNotFound: Space not existed! And drop the used space Scenario: Clear space function test diff --git a/tests/tck/features/optimizer/PrunePropertiesRule.feature b/tests/tck/features/optimizer/PrunePropertiesRule.feature index 1367c42f10e..e5eba0613e3 100644 --- a/tests/tck/features/optimizer/PrunePropertiesRule.feature +++ b/tests/tck/features/optimizer/PrunePropertiesRule.feature @@ -851,3 +851,30 @@ Feature: Prune Properties rule | __NULL__ | __NULL__ | NULL | NULL | NULL | __NULL__ | | __NULL__ | __NULL__ | NULL | NULL | NULL | __NULL__ | Then drop the used space + + Scenario: Project on not exist tag + Given a graph with space named "nba" + When executing query: + """ + MATCH (v:player)-[e:like]->(t) WHERE v.player.name=='Tim Duncan' RETURN v.player.name, v.x.y, v.player.age + """ + Then the result should be, in order, with relax comparison: + | v.player.name | v.x.y | v.player.age | + | "Tim Duncan" | __NULL__ | 42 | + | "Tim Duncan" | __NULL__ | 42 | + When executing query: + """ + MATCH (v:player)-[:like]->(t) WHERE v.player.name=="Tim Duncan" RETURN v.player.name, properties(v), t + """ + Then the result should be, in order, with relax comparison: + | v.player.name | properties(v) | t | + | "Tim Duncan" | {age: 42, name: "Tim Duncan", speciality: "psychology"} | ("Tony Parker" :player{age: 36, name: "Tony Parker"}) | + | "Tim Duncan" | {age: 42, name: "Tim Duncan", speciality: "psychology"} | ("Manu Ginobili" :player{age: 41, name: "Manu Ginobili"}) | + When executing query: + """ + MATCH (v:player)-[:like]->(t) WHERE v.player.name=="Tim Duncan" RETURN v.player.name, t.errortag.name, properties(v), t + """ + Then the result should be, in order, with relax comparison: + | v.player.name | t.errortag.name | properties(v) | t | + | "Tim Duncan" | __NULL__ | {age: 42, name: "Tim Duncan", speciality: "psychology"} | ("Tony Parker" :player{age: 36, name: "Tony Parker"}) | + | "Tim Duncan" | __NULL__ | {age: 42, name: "Tim Duncan", speciality: "psychology"} | ("Manu Ginobili" :player{age: 41, name: "Manu Ginobili"}) | diff --git a/tests/tck/features/schema/Schema.feature b/tests/tck/features/schema/Schema.feature index 986ac30b16d..d06bc0bad9c 100644 --- a/tests/tck/features/schema/Schema.feature +++ b/tests/tck/features/schema/Schema.feature @@ -96,7 +96,7 @@ Feature: Insert string vid of vertex and edge """ DESCRIBE TAG not_exist """ - Then a ExecutionError should be raised at runtime: Tag not existed! + Then a ExecutionError should be raised at runtime: TagNotFound: Tag not existed! # unreserved keyword When executing query: """ @@ -193,7 +193,7 @@ Feature: Insert string vid of vertex and edge """ DROP TAG not_exist_tag """ - Then a ExecutionError should be raised at runtime: Tag not existed! + Then a ExecutionError should be raised at runtime: TagNotFound: Tag not existed! # drop if exists with not exist tag When executing query: """ @@ -300,7 +300,7 @@ Feature: Insert string vid of vertex and edge """ DESCRIBE EDGE not_exist """ - Then a ExecutionError should be raised at runtime: Edge not existed! + Then a ExecutionError should be raised at runtime: EdgeNotFound: Edge not existed! # create edge with timestamp When executing query: """ @@ -378,7 +378,7 @@ Feature: Insert string vid of vertex and edge """ DROP EDGE not_exist_edge """ - Then a ExecutionError should be raised at runtime: Edge not existed! + Then a ExecutionError should be raised at runtime: EdgeNotFound: Edge not existed! # drop if exists When executing query: """ diff --git a/tests/tck/features/yield/NoSpaceChosen.feature b/tests/tck/features/yield/NoSpaceChosen.feature index e83140b13fb..229e571deb7 100644 --- a/tests/tck/features/yield/NoSpaceChosen.feature +++ b/tests/tck/features/yield/NoSpaceChosen.feature @@ -88,12 +88,12 @@ Feature: Yield """ YIELD $$.dummyTag.p """ - Then a ExecutionError should be raised at runtime: TagName `dummyTag' is nonexistent + Then a ExecutionError should be raised at runtime: TagNotFound: TagName `dummyTag` When executing query: """ YIELD $^.dummyTag.p """ - Then a ExecutionError should be raised at runtime: TagName `dummyTag' is nonexistent + Then a ExecutionError should be raised at runtime: TagNotFound: TagName `dummyTag` When executing query: """ YIELD $-.dummyTag.p diff --git a/tests/tck/features/yield/return.feature b/tests/tck/features/yield/return.feature index d7ebe9fbf12..d18cd814ea8 100644 --- a/tests/tck/features/yield/return.feature +++ b/tests/tck/features/yield/return.feature @@ -62,12 +62,12 @@ Feature: Return """ RETURN $$.dummyTag.p """ - Then a ExecutionError should be raised at runtime: TagName `dummyTag' is nonexistent + Then a ExecutionError should be raised at runtime: TagNotFound: TagName `dummyTag` When executing query: """ RETURN $^.dummyTag.p """ - Then a ExecutionError should be raised at runtime: TagName `dummyTag' is nonexistent + Then a ExecutionError should be raised at runtime: TagNotFound: TagName `dummyTag` When executing query: """ RETURN $-.dummyTag.p diff --git a/tests/tck/features/yield/yield.IntVid.feature b/tests/tck/features/yield/yield.IntVid.feature index 6fa0c5bd873..0aef1b6b7e6 100644 --- a/tests/tck/features/yield/yield.IntVid.feature +++ b/tests/tck/features/yield/yield.IntVid.feature @@ -298,12 +298,12 @@ Feature: Yield Sentence """ $var = GO FROM hash("Boris Diaw") OVER serve YIELD $^.player.name AS name, serve.start_year AS start, $$.team.name AS team;YIELD $$.a.team """ - Then a ExecutionError should be raised at runtime: TagName `a' is nonexistent + Then a ExecutionError should be raised at runtime: TagNotFound: TagName `a` When executing query: """ $var = GO FROM hash("Boris Diaw") OVER serve YIELD $^.player.name AS name, serve.start_year AS start, $$.team.name AS team;YIELD $^.a.team """ - Then a ExecutionError should be raised at runtime: TagName `a' is nonexistent + Then a ExecutionError should be raised at runtime: TagNotFound: TagName `a` When executing query: """ $var = GO FROM hash("Boris Diaw") OVER serve YIELD $^.player.name AS name, serve.start_year AS start, $$.team.name AS team;YIELD a.team diff --git a/tests/tck/features/yield/yield.feature b/tests/tck/features/yield/yield.feature index 71afde05e15..47677b2cb3e 100644 --- a/tests/tck/features/yield/yield.feature +++ b/tests/tck/features/yield/yield.feature @@ -308,12 +308,12 @@ Feature: Yield Sentence """ $var = GO FROM "Boris Diaw" OVER serve YIELD $^.player.name AS name, serve.start_year AS start, $$.team.name AS team;YIELD $$.a.team """ - Then a ExecutionError should be raised at runtime: TagName `a' is nonexistent + Then a ExecutionError should be raised at runtime: TagNotFound: TagName `a` When executing query: """ $var = GO FROM "Boris Diaw" OVER serve YIELD $^.player.name AS name, serve.start_year AS start, $$.team.name AS team;YIELD $^.a.team """ - Then a ExecutionError should be raised at runtime: TagName `a' is nonexistent + Then a ExecutionError should be raised at runtime: TagNotFound: TagName `a` When executing query: """ $var = GO FROM "Boris Diaw" OVER serve YIELD $^.player.name AS name, serve.start_year AS start, $$.team.name AS team;YIELD a.team