Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

format go yield #2944

Merged
merged 9 commits into from
Sep 30, 2021
Merged

format go yield #2944

merged 9 commits into from
Sep 30, 2021

Conversation

nevermore3
Copy link
Contributor

@nevermore3 nevermore3 commented Sep 27, 2021

$^ represents the starting point of the edge
$$ represents the destination point of the edge

go from 'a' over * yield $^ as startNode, $$ as endNode, edge as relationship

the type of $^ and $$ is VERTEX, must add alias when use $^ or $$

@nevermore3 nevermore3 added doc affected PR: improvements or additions to documentation wip Solution: work in progress labels Sep 27, 2021
@nevermore3
Copy link
Contributor Author

subjob of #2594

@Sophie-Xie Sophie-Xie added this to the v2.6.0 milestone Sep 27, 2021
@nevermore3 nevermore3 force-pushed the format_go branch 11 times, most recently from 41066e3 to f52cee7 Compare September 29, 2021 13:11
@nevermore3 nevermore3 added ready-for-testing PR: ready for the CI test and removed wip Solution: work in progress labels Sep 29, 2021
src/parser/parser.yy Outdated Show resolved Hide resolved
src/graph/validator/test/QueryValidatorTest.cpp Outdated Show resolved Hide resolved
tests/tck/features/go/GoYieldVertexEdge.feature Outdated Show resolved Hide resolved
src/graph/validator/GoValidator.cpp Outdated Show resolved Hide resolved
src/graph/validator/GoValidator.cpp Outdated Show resolved Hide resolved
src/graph/validator/GoValidator.cpp Outdated Show resolved Hide resolved
CPWstatic
CPWstatic previously approved these changes Sep 30, 2021
src/graph/validator/GoValidator.cpp Outdated Show resolved Hide resolved
for (auto* col : goCtx_->yieldExpr->columns()) {
auto* vertexExpr = ExpressionUtils::findAny(col->expr(), {Expression::Kind::kVertex});
if (vertexExpr != nullptr) {
const auto& colName = static_cast<const VertexExpression*>(vertexExpr)->name();
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if (existExpr.count(colName)==0) {
  if (colName == SRC) {

  } else if (colName == DST) {}
  existExpr[colName] = true;
}

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fix in other PR

@CPWstatic CPWstatic merged commit 8450ba9 into vesoft-inc:master Sep 30, 2021
@nevermore3 nevermore3 deleted the format_go branch September 30, 2021 09:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc affected PR: improvements or additions to documentation ready-for-testing PR: ready for the CI test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants