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

Reorg leader table with graph ci fix #439

Merged
merged 2 commits into from
Apr 15, 2021
Merged

Reorg leader table with graph ci fix #439

merged 2 commits into from
Apr 15, 2021

Conversation

liuyu85cn
Copy link
Contributor

No description provided.

@liuyu85cn liuyu85cn added ready-for-testing PR: ready for the CI test Ready-to-review labels Apr 15, 2021
panda-sheep
panda-sheep previously approved these changes Apr 15, 2021
Copy link
Contributor

@panda-sheep panda-sheep left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

lionel.liu@vesoft.com and others added 2 commits April 15, 2021 12:09
* encode/decode in meta works

simplify leader val to Host and term

show hosts works

manual test passed

clear some code

do some clean

restore meta test

all can compile, ready to go

fix some test

so dome clean

do some clean

* fix compile error

* clean some log

* do some clean

* address comment

* address some comments

* address comments

Co-authored-by: CBS <56461666+bright-starry-sky@users.noreply.github.com>
@critical27 critical27 merged commit f3d3ba7 into vesoft-inc:master Apr 15, 2021
@liuyu85cn liuyu85cn deleted the reorg-leader-table-with-graph-ci-fix branch April 15, 2021 07:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
ready-for-testing PR: ready for the CI test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants