Skip to content

Commit 89cd019

Browse files
fix(interactive): Fix group by with vertex as group key (#4482)
As titled. Fix #4480 --------- Co-authored-by: liulx20 <[email protected]>
1 parent 7b8a4aa commit 89cd019

File tree

2 files changed

+5
-0
lines changed

2 files changed

+5
-0
lines changed

flex/engines/graph_db/runtime/execute/ops/retrieve/group_by.cc

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -991,6 +991,10 @@ bl::result<ReadOpBuildResultT> GroupByOprBuilder::Build(
991991
int idx = 0;
992992
for (const auto& var : vars) {
993993
auto alias = mappings[idx++].second;
994+
if (!var.has_property()) {
995+
continue;
996+
}
997+
994998
Var var_(graph, ctx, var, VarType::kPathVar);
995999
if (var_.type() == RTAnyType::kStringValue) {
9961000
TypedKeyCollector<std::string_view>::TypedKeyWrapper wrapper(

flex/interactive/sdk/python/gs_interactive/tests/test_robustness.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,7 @@
5555
"MATCH(a)-[b]->(c) return count(b)",
5656
"MATCH(a)-[b]->(c) return b",
5757
"MATCH(a)-[b]->(c) return c.id",
58+
"MATCH(a)-[b]->(c) return count(c), c, c.id;",
5859
]
5960

6061

0 commit comments

Comments
 (0)