Skip to content
This repository has been archived by the owner on Feb 6, 2024. It is now read-only.

refactor: refactor cluster package #98

Merged
merged 3 commits into from
Nov 9, 2022

Conversation

chunshao90
Copy link
Contributor

@chunshao90 chunshao90 commented Nov 8, 2022

Which issue does this PR close?

Refer to #94

Rationale for this change

What changes are included in this PR?

Extract cluster metadata management into a separate module, simplifying cluster module operations.

Are there any user-facing changes?

How does this change test

Old tests have covered.

@chunshao90 chunshao90 marked this pull request as ready for review November 8, 2022 13:19
server/cluster/manager.go Outdated Show resolved Hide resolved
server/cluster/manager.go Outdated Show resolved Hide resolved
server/cluster/manager.go Outdated Show resolved Hide resolved
server/cluster/manager.go Outdated Show resolved Hide resolved
server/cluster/manager.go Outdated Show resolved Hide resolved
server/coordinator/procedure/scatter.go Outdated Show resolved Hide resolved
server/coordinator/procedure/create_table.go Outdated Show resolved Hide resolved
server/coordinator/scheduler.go Outdated Show resolved Hide resolved
server/coordinator/scheduler.go Outdated Show resolved Hide resolved
server/service/grpc/service.go Outdated Show resolved Hide resolved
@ZuLiangWang ZuLiangWang closed this Nov 9, 2022
@ZuLiangWang ZuLiangWang reopened this Nov 9, 2022
Copy link
Contributor

@ZuLiangWang ZuLiangWang left a comment

Choose a reason for hiding this comment

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

LGTM

@ZuLiangWang ZuLiangWang merged commit e0002ff into apache:main Nov 9, 2022
ZuLiangWang pushed a commit to ZuLiangWang/ceresmeta that referenced this pull request Nov 25, 2022
* refactor: refactor cluster package

* refactor code

* refactor by CR
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants