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

The default order of LOOKUP in 2.6.0 differs from that in 2.5.1 #3244

Closed
randomJoe211 opened this issue Nov 1, 2021 · 2 comments
Closed
Assignees
Labels
incompatible PR: incompatible with the recently released version
Milestone

Comments

@randomJoe211
Copy link
Contributor

When running LOOKUP ON player WHERE player.age > 40; in 2.6.0, the result is always:

> LOOKUP ON player WHERE player.age  > 40;
+-------------+
| VertexID    |
+-------------+
| "player100" |
| "player125" |
| "player127" |
| "player136" |
| "player140" |
| "player141" |
| "player144" |
| "player148" |
+-------------+

But in 2.5.1, the result is always:

> LOOKUP ON player WHERE player.age  > 40;
+-------------+
| VertexID    |
+-------------+
| "player100" |
+-------------+
| "player148" |
+-------------+
| "player144" |
+-------------+
| "player141" |
+-------------+
| "player140" |
+-------------+
| "player136" |
+-------------+
| "player127" |
+-------------+
| "player125" |
+-------------+

Both have 8 rows, but the orders are totally different. Is this the same issue as #3242?

@randomJoe211 randomJoe211 added the incompatible PR: incompatible with the recently released version label Nov 1, 2021
@Sophie-Xie Sophie-Xie added this to the v3.0.0 milestone Nov 1, 2021
@Sophie-Xie Sophie-Xie added type/bug Type: something is unexpected and removed type/bug Type: something is unexpected labels Nov 1, 2021
@cangfengzhs
Copy link
Contributor

I think if the "order by" clause is not given, any order is correct as long as the returned result set is correct.

@CPWstatic
Copy link
Contributor

I had explain this for several times, the order of results is not guaranteed since the data are pulling from the distributed system. I'll close this issue first, and the #3242 one. Please talk to me offline if you are truly troubled by this senario.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
incompatible PR: incompatible with the recently released version
Projects
None yet
Development

No branches or pull requests

4 participants