Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(docs): Fix two typos in design.md #3445

Merged
merged 1 commit into from
Nov 16, 2020

Conversation

haslersn
Copy link
Contributor

  • I added CHANGELOG entry for this change.
  • Change is not relevant to the end user.

Changes

  • "scaled up" -> "scaled out"
  • "as approximately as" -> "approximately as"

Copy link
Member

@bwplotka bwplotka left a comment

Choose a reason for hiding this comment

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

LGTM (: Can you fix DCO?

@@ -138,7 +138,7 @@ The compactor also does additional batch processing such as down-sampling and ap

## Scaling

None of the Thanos components provides any means of sharding. The only explicitly scalable component are query nodes, which are stateless and can be scaled up arbitrarily. Scaling of storage capacity is ensured by relying on an external object storage system.
None of the Thanos components provides any means of sharding. The only explicitly scalable component are query nodes, which are stateless and can be scaled out arbitrarily. Scaling of storage capacity is ensured by relying on an external object storage system.
Copy link
Member

Choose a reason for hiding this comment

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

None of the Thanos components provides any means of sharding.

Wow, this does not hold truth anymore ;p

I think at this point we need to rename it as Initial Design? (:
cc @brancz

@haslersn
Copy link
Contributor Author

Can you fix DCO?

What's DCO?

@soniasingla
Copy link
Contributor

@haslersn check your tests are failing, visit here The instructions are written how to sign it correctly!

And to read about DCO, check this out, hope it helps you :)

@haslersn haslersn force-pushed the fix-docs-design-typo branch from 6c75826 to 0067d04 Compare November 15, 2020 17:07
Copy link
Member

@bwplotka bwplotka left a comment

Choose a reason for hiding this comment

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

Thanks!

And thanks @soniasingla for helping out!

@bwplotka bwplotka merged commit 755b275 into thanos-io:master Nov 16, 2020
Oghenebrume50 pushed a commit to Oghenebrume50/thanos that referenced this pull request Dec 7, 2020
Signed-off-by: haslersn <[email protected]>
Signed-off-by: Oghenebrume50 <[email protected]>
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