-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[doc]Add return doc. #1246
[doc]Add return doc. #1246
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thank you for this brilliant new function and doc. I'll take care of the minor errors.
|
||
# 示例 | ||
|
||
``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
````ngql`
# 示例 | ||
|
||
``` | ||
(user@127.0.0.1) [myspace_test2]> $A = GO FROM 200 OVER like YIELD like._dst AS dst; \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nebula>...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree
(user@127.0.0.1) [myspace_test2] is too long & meanless to read. can you just modify to
nebula>
# 示例 | ||
|
||
``` | ||
(user@127.0.0.1) [myspace_test2]> $A = GO FROM 200 OVER like YIELD like._dst AS dst; \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree
(user@127.0.0.1) [myspace_test2] is too long & meanless to read. can you just modify to
nebula>
# 示例 | ||
|
||
``` | ||
(user@127.0.0.1) [myspace_test2]> $A = GO FROM 200 OVER like YIELD like._dst AS dst; \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what is your style for variable? Upper case or lower case?
why $A and $rA?
Return 语句用于返回条件成立时的结果。 | ||
|
||
```ngql | ||
RETURN <var_ref> IF <var_ref> IS NOT NULL |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do you support IS NULL also? or only IS NOT NULL.
-> $rA = YIELD $A.* WHERE $A.dst == 201; \ | ||
-> RETURN $rA IF $rA is NOT NULL; \ /* return here since $rA is not empty */ | ||
-> GO FROM $A.dst OVER like; /* will never be executed*/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
-> remove spaces and ->
indent
Jenkins go! |
Unit testing passed. |
1 similar comment
Unit testing passed. |
* Nebula Graph to NebulaGraph * fix ctest * update nebula * fix comment * fix comment * fix comment Co-authored-by: Sophie <84560950+Sophie-Xie@users.noreply.github.com>
No description provided.