Skip to content
This repository has been archived by the owner on Jan 10, 2019. It is now read-only.

Add docs for multi-dc support #130

Merged
merged 1 commit into from
Aug 7, 2015
Merged

Add docs for multi-dc support #130

merged 1 commit into from
Aug 7, 2015

Conversation

BenWhitehead
Copy link
Contributor

No description provided.

@BenWhitehead
Copy link
Contributor Author

@ssk2 or @joel-hamill can I get some feedback on this new document?

@ssk2
Copy link
Contributor

ssk2 commented Aug 6, 2015

I've created an internal JIRA (DOCS-118) for @mattpennathe3rd and @joel-hamill.


## How it works

Cassandra on Mesos allows an operator to configure the framework to provide the Datacenter and rack level information
Copy link
Contributor

Choose a reason for hiding this comment

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

Datacenter -> datacenter

@BenWhitehead
Copy link
Contributor Author

Updates incorporated


# Multi Datacenter

As of version 0.2.0 Cassandra on Mesos allows for providing configuration
Copy link
Contributor

Choose a reason for hiding this comment

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

This sentence is hard to parse.

@BenWhitehead
Copy link
Contributor Author

Thanks for the review @ssk2 and @joel-hamill.

Updates in place. I'll squash before merging them.

@mattpennathe3rd
Copy link

@BenWhitehead @joel-hamill Made my own pass through this and made a number of changes. Please let me know how this looks and if my method of proposing/committing these changes was improper.

@ssk2
Copy link
Contributor

ssk2 commented Aug 7, 2015

@mattpennathe3rd perfectly acceptable method of reviewing :)

@BenWhitehead
Copy link
Contributor Author

@mattpennathe3rd Thanks for the feedback. I've made only a couple modifications to your proposal specifically to make sure that when we're talking about Cassandra on Mesos we use the full name so that people don't think it's about cassandra itself.

@BenWhitehead
Copy link
Contributor Author

going to rebase then merge. Thanks @ssk2 @joel-hamill @mattpennathe3rd

BenWhitehead added a commit that referenced this pull request Aug 7, 2015
@BenWhitehead BenWhitehead merged commit e8fac0a into v0.2.0-rc Aug 7, 2015
@BenWhitehead BenWhitehead deleted the multi-dc-docs branch August 7, 2015 23:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants