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

Add cluster-mode design document #150

Merged
merged 3 commits into from
Jul 18, 2021
Merged

Add cluster-mode design document #150

merged 3 commits into from
Jul 18, 2021

Conversation

hackerwins
Copy link
Member

What this PR does / why we need it:

Writing a design document for Cluster-Mode.

Which issue(s) this PR fixes:

Fixes #11

Special notes for your reviewer:

Does this PR introduce a user-facing change?:

Additional documentation:

Checklist:

  • Added relevant tests or not required
  • Didn't break anything

@hackerwins hackerwins added wip 🏗 documentation 📔 Improvements or additions to documentation labels Feb 16, 2021
@codecov
Copy link

codecov bot commented Feb 16, 2021 •
edited
Loading

Codecov Report

Merging #150 (a9ef5c9) into main (3201c71) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #150   +/-   ##
=======================================
  Coverage   57.85%   57.85%           
=======================================
  Files          44       44           
  Lines        3732     3732           
=======================================
  Hits         2159     2159           
  Misses       1374     1374           
  Partials      199      199
Impacted Files Coverage Δ
yorkie/config.go 89.28% <0.00%> (ø)
yorkie/backend/backend.go 11.11% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3201c71...a9ef5c9. Read the comment docs.

@hackerwins hackerwins force-pushed the design-cluster-mode branch 6 times, most recently from 5f76454 to 43a7323 Compare February 22, 2021 08:53
@hackerwins hackerwins marked this pull request as draft March 19, 2021 01:21
hackerwins
hackerwins commented Mar 30, 2021
View reviewed changes
design/cluster-mode.md Outdated Show resolved Hide resolved
hackerwins
hackerwins commented Jun 2, 2021
View reviewed changes
design/cluster-mode.md Outdated Show resolved Hide resolved
@hackerwins hackerwins force-pushed the design-cluster-mode branch 2 times, most recently from a5d5600 to 34e9732 Compare July 17, 2021 02:16
@hackerwins hackerwins marked this pull request as ready for review July 17, 2021 02:17
@hackerwins hackerwins assigned mojosoeun and ppeeou and unassigned hackerwins Jul 17, 2021
@hackerwins hackerwins requested a review from a team July 17, 2021 02:17
@hackerwins hackerwins force-pushed the design-cluster-mode branch from 34e9732 to a9ef5c9 Compare July 17, 2021 05:30
dc7303
dc7303 approved these changes Jul 17, 2021
View reviewed changes
Copy link
Member

@dc7303 dc7303 left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@dc7303 dc7303 removed their assignment Jul 17, 2021
@mojosoeun mojosoeun removed their assignment Jul 17, 2021
ppeeou
ppeeou approved these changes Jul 18, 2021
View reviewed changes
Copy link
Member

@ppeeou ppeeou left a comment

Choose a reason for hiding this comment

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

LGTM

@hackerwins hackerwins merged commit 89b549c into main Jul 18, 2021
@hackerwins hackerwins deleted the design-cluster-mode branch July 18, 2021 08:50
jeonjonghyeok pushed a commit to jeonjonghyeok/yorkie that referenced this pull request Aug 4, 2022
@hackerwins @jeonjonghyeok
Add cluster-mode design document (yorkie-team#150)
1b12eba
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dc7303 dc7303 dc7303 approved these changes

@ppeeou ppeeou ppeeou approved these changes

@mojosoeun mojosoeun mojosoeun approved these changes

Assignees

@ppeeou ppeeou

Labels
documentation 📔 Improvements or additions to documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Support HA servers in active-active manner
4 participants
@hackerwins @mojosoeun @ppeeou @dc7303