Skip to content

docs: clustered index proposal#17044

Merged
bb7133 merged 3 commits into
pingcap:masterfrom
coocood:cluster-index-proposal
May 11, 2020
Merged

docs: clustered index proposal#17044
bb7133 merged 3 commits into
pingcap:masterfrom
coocood:cluster-index-proposal

Conversation

@coocood
Copy link
Copy Markdown
Member

@coocood coocood commented May 8, 2020

What problem does this PR solve?

Problem Summary:
Add a proposal doc for the clustered index feature.

Release note

  • No release note

@sre-bot
Copy link
Copy Markdown
Contributor

sre-bot commented May 8, 2020

@coocood
Copy link
Copy Markdown
Member Author

coocood commented May 8, 2020

/rebuild

@coocood coocood requested a review from wjhuang2016 May 8, 2020 10:40
@codecov
Copy link
Copy Markdown

codecov Bot commented May 8, 2020

Codecov Report

Merging #17044 into master will not change coverage.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             master     #17044   +/-   ##
===========================================
  Coverage   80.0256%   80.0256%           
===========================================
  Files           510        510           
  Lines        138993     138993           
===========================================
  Hits         111230     111230           
  Misses        18807      18807           
  Partials       8956       8956           

Comment thread docs/design/2020-05-08-cluster-index.md
Comment thread docs/design/2020-05-08-cluster-index.md
Comment thread docs/design/2020-05-08-cluster-index.md
Comment thread docs/design/2020-05-08-cluster-index.md
Comment thread docs/design/2020-05-08-cluster-index.md
Comment thread docs/design/2020-05-08-cluster-index.md
Copy link
Copy Markdown
Contributor

@jackysp jackysp left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Member

@wjhuang2016 wjhuang2016 left a comment

Choose a reason for hiding this comment

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

LGTM

coocood added a commit to coocood/parser that referenced this pull request May 9, 2020
@Deardrops Deardrops self-requested a review May 9, 2020 10:26
Comment thread docs/design/2020-05-08-cluster-index.md
Comment thread docs/design/2020-05-08-cluster-index.md
Comment thread docs/design/2020-05-08-cluster-index.md
Copy link
Copy Markdown
Member

@bb7133 bb7133 left a comment

Choose a reason for hiding this comment

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

LGTM

@bb7133 bb7133 merged commit 6b6be0b into pingcap:master May 11, 2020
@coocood coocood deleted the cluster-index-proposal branch June 1, 2020 11:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants