Skip to content

Conversation

@bookshelfdave
Copy link
Contributor

No description provided.

Copy link
Contributor

Choose a reason for hiding this comment

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

This file doesn't exist in the navigation

@coderoshi
Copy link
Contributor

I hope my comments aren't too pedantic. Just trying to keep consistency in style. These docs are amazing, thanks for all the hard work.

Copy link
Contributor

Choose a reason for hiding this comment

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

These lines are pretty pedantic.

* connect the source cluster to the sink cluster

On any node in the source cluster:

`riak-repl connect <sink_cluster_name>`

For example:

  riak-repl connect boston

where <sink_cluster_name> is replaced with the name of the sink cluster.

Could be just as clear with:

* connect the source cluster to the sink cluster. Assuming the sink cluster name *boston*:

  `riak-repl connect boston`

You've been using boston as the sink cluster all along, no need to change it now.

Copy link
Contributor

Choose a reason for hiding this comment

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

don't forget TODO

Copy link
Contributor

Choose a reason for hiding this comment

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

1.3.0+ ... version 3 isn't that old :)

@coderoshi
Copy link
Contributor

Will +1 when UpgradeV2toV3 formatting is fixed

@coderoshi
Copy link
Contributor

+1 bravo bro

bookshelfdave pushed a commit that referenced this pull request Jul 2, 2013
@bookshelfdave bookshelfdave merged commit c1fcb78 into master Jul 2, 2013
@bookshelfdave bookshelfdave deleted the dip_enterprise14 branch February 26, 2014 14:43
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.

5 participants