Skip to content
This repository has been archived by the owner on Dec 1, 2022. It is now read-only.

add go step limit OR sample (validate) #989

Merged
merged 5 commits into from
Jun 22, 2021

Conversation

nevermore3
Copy link
Contributor

@nevermore3 nevermore3 commented Apr 27, 2021

support setting different limit or sample for each step in GO
For example:
GO 3 STEPS FROM OVER WHERE [filter1, filter2, filter3] YIELD LIMIT [n1, n2, n3]
GO 3 STEPS FROM OVER WHERE [filter1, filter2, filter3] YIELD SAMPLE [n1, n2, n3]

As well as WHERE Clause

@nevermore3 nevermore3 added wip Solution: work in progress ready-for-testing PR: ready for the CI test and removed wip Solution: work in progress ready-for-testing PR: ready for the CI test labels Apr 27, 2021
@nevermore3 nevermore3 force-pushed the go_steps_limit branch 4 times, most recently from 19fd22e to acfd478 Compare April 29, 2021 08:32
@nevermore3 nevermore3 added ready-for-testing PR: ready for the CI test and removed wip Solution: work in progress labels Apr 29, 2021
@nevermore3 nevermore3 changed the title add go step limit OR sample add go step limit OR sample (validate) Apr 29, 2021
@Shylock-Hg
Copy link
Contributor

Could you explain the meaning of new syntax?

@nevermore3
Copy link
Contributor Author

Could you explain the meaning of new syntax?

In the first step of expansion, I only take 10 edges, and in the second step of expansion, I only take 30 edges, I can use the following syntax

go 2 steps from "Tim Duncan" over like limit [10, 30], same with SAMPLE

@yixinglu yixinglu merged commit 3c65733 into vesoft-inc:master Jun 22, 2021
nevermore3 added a commit to nevermore3/nebula-graph that referenced this pull request Jun 24, 2021
* add go step limit OR sample

* remove map case

* validate truncate

* fix error

* return error
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
med-pri Priority: medium new feature Type: new feature ready-for-testing PR: ready for the CI test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants