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

remove redundant project&dedup in findpath & fix path error #912

Merged
merged 6 commits into from
Apr 2, 2021

Conversation

nevermore3
Copy link
Contributor

@nevermore3 nevermore3 commented Apr 1, 2021

image

@nevermore3 nevermore3 added the wip Solution: work in progress label Apr 1, 2021
@nevermore3 nevermore3 changed the title remove redundant project&dedup in findpath remove redundant project&dedup in findpath & fix path error Apr 1, 2021
@nevermore3 nevermore3 added ready-for-review ready-for-testing PR: ready for the CI test and removed wip Solution: work in progress labels Apr 1, 2021
@nevermore3 nevermore3 requested a review from a team April 1, 2021 10:53
@nevermore3 nevermore3 added the bug Type: something is unexpected label Apr 2, 2021
@yixinglu yixinglu merged commit ad355b5 into vesoft-inc:master Apr 2, 2021
jude-zhu pushed a commit to jude-zhu/nebula-graph that referenced this pull request Apr 2, 2021
…nc#912)

* remove redundant project&dedup in findpath

* fix path error & add testcase

* add testcase

Co-authored-by: Yee <2520865+yixinglu@users.noreply.github.com>
(cherry picked from commit ad355b5)
@nevermore3 nevermore3 deleted the fix_path_error branch April 6, 2021 03:10
yixinglu added a commit that referenced this pull request Apr 6, 2021
* Remove redundant project&dedup in findpath & fix path error (#912)

* remove redundant project&dedup in findpath

* fix path error & add testcase

* add testcase

Co-authored-by: Yee <2520865+yixinglu@users.noreply.github.com>
(cherry picked from commit ad355b5)

* Fix delete vertex check wrong vid type (#917)

* fix delete vertex use ref vid

* fix fmt

(cherry picked from commit 67523d8)

Co-authored-by: jimingquan <mingquan.ji@vesoft.com>
Co-authored-by: laura-ding <48548375+laura-ding@users.noreply.github.com>
Co-authored-by: Yee <2520865+yixinglu@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Type: something is unexpected ready-for-testing PR: ready for the CI test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants