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.

Sign up for GitHub

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

Jump to bottom

Fix the dangle limit crash. #2558

Closed
wants to merge 2 commits into from
Closed

Fix the dangle limit crash. #2558

wants to merge 2 commits into from

Conversation

Shylock-Hg
Copy link
Contributor

Close #2548

@Shylock-Hg Shylock-Hg added ready-for-testing PR: ready for the CI test type/bug Type: something is unexpected labels Aug 24, 2021
@Shylock-Hg Shylock-Hg requested a review from a team August 24, 2021 06:39
@Shylock-Hg Shylock-Hg requested a review from a team August 25, 2021 06:25
@jievince
Copy link
Contributor

I think it has been fixed in vesoft-inc/nebula-graph#1314,
I will move it to current repository

@Shylock-Hg
Copy link
Contributor Author

I think it has been fixed in vesoft-inc/nebula-graph#1314,
I will move it to current repository

Ok, please add this test case.

@Shylock-Hg Shylock-Hg closed this Aug 25, 2021
@Shylock-Hg Shylock-Hg deleted the fix/dangle-limit branch February 9, 2022 07:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Aiee Aiee Aiee approved these changes

Assignees
No one assigned
Labels
ready-for-testing PR: ready for the CI test type/bug Type: something is unexpected
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Crush when using LIMIT after ;
3 participants
@Shylock-Hg @jievince @Aiee