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

fix unstable job related ut #2776

Merged
merged 5 commits into from
Sep 6, 2021
Merged

Conversation

critical27
Copy link
Contributor

JobId + TaskId is not unique in some ut, which may cause isFinished return earlier than expected.

Close #2761

@critical27 critical27 added the ready-for-testing PR: ready for the CI test label Sep 1, 2021
@critical27 critical27 removed the ready-for-testing PR: ready for the CI test label Sep 1, 2021
panda-sheep
panda-sheep previously approved these changes Sep 1, 2021
@bright-starry-sky bright-starry-sky merged commit b5b6d89 into vesoft-inc:master Sep 6, 2021
@critical27 critical27 deleted the fix branch September 6, 2021 03:20
yixinglu added a commit to yixinglu/nebula that referenced this pull request Sep 14, 2023
…esoft-inc#2776)

* Support variable when seeking by id in match clause vesoft-inc#5486

* Support variable when seeking by property index in match clause (vesoft-inc#5553)

* Support variable when seeking by prop index in match clause

* Rename

* Move the index query context construction outside of OptRule

* Move the optimizer utils to common graph util folder

* Eval argument values in index scan executor

* Fix bug

* refactor and fix error

* Update tck tests

---------

Co-authored-by: Sophie <[email protected]>

* Resolve the conflicts

---------

Co-authored-by: Yee <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-for-testing PR: ready for the CI test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

UT index_ttl_test is not stable
3 participants