Skip to content

op-guide: add clustering.#4

Merged
siddontang merged 11 commits into
masterfrom
siddontang/clustering
Jul 29, 2016
Merged

op-guide: add clustering.#4
siddontang merged 11 commits into
masterfrom
siddontang/clustering

Conversation

@siddontang
Copy link
Copy Markdown
Member

@shenli @queenypingcap

@siddontang siddontang changed the title op-guide: add clustering. [DNM] op-guide: add clustering. Jul 26, 2016
Comment thread op-guide/clustering.md Outdated
```bash
tidb-server --store=tikv \
--path="127.0.0.1:2379/pd?cluster=1"
-P 5001
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Why not use 4000? It is our official port.

@siddontang
Copy link
Copy Markdown
Member Author

PTAL @shenli @iamxy

Comment thread op-guide/docker.md Outdated
```

+ Use `docker-compose up -d` to create and start the cluster.
+ Use `docker-compose port tidb 4000` to print the TiDB host port. For example, if the output is `0.0.0.0:32966`, the TiDB host port is `32966`.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

What's mean by "host port"?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

docker-compose port -h
Print the public port for a port binding.

use public port instead? @queenypingcap

@shenli
Copy link
Copy Markdown
Member

shenli commented Jul 29, 2016

LGTM

1 similar comment
@QueenyJin
Copy link
Copy Markdown
Contributor

LGTM

@siddontang siddontang changed the title [DNM] op-guide: add clustering. op-guide: add clustering. Jul 29, 2016
@siddontang siddontang merged commit 344d720 into master Jul 29, 2016
@siddontang siddontang deleted the siddontang/clustering branch August 12, 2016 06:26
terry1purcell added a commit to terry1purcell/docs that referenced this pull request Apr 2, 2024
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.

3 participants