Skip to content

First version of a doc about API scope.#2642

Merged
alainjobart merged 1 commit intovitessio:masterfrom
alainjobart:doc
Mar 20, 2017
Merged

First version of a doc about API scope.#2642
alainjobart merged 1 commit intovitessio:masterfrom
alainjobart:doc

Conversation

@alainjobart
Copy link
Copy Markdown
Contributor

@sougou first draft, let's talk about it, or you can comment.

Copy link
Copy Markdown
Contributor

@sougou sougou left a comment

Choose a reason for hiding this comment

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

Looks like it covers everything we discussed. The scope of a shard query is still being debated. I can add the details once that's finalized.

doc/APIScope.md Outdated
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The 0 shard is a special case which will always create confusion. Hence I suggest to better call it out e.g.

For unsharded keyspaces, or custom sharded keyspaces, the shard names have traditionally been numbers e.g. 0. (It is important to note that Vitess does not interpret these shards (including shard name 0) as range-based shard.)

@alainjobart alainjobart merged commit 7293a02 into vitessio:master Mar 20, 2017
@alainjobart alainjobart deleted the doc branch March 20, 2017 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants