Skip to content

Conversation

@AlexanderSaydakov
Copy link
Contributor

Relevant pull requests (aggregators):
apache/druid#5148
apache/druid#5002

@fjy
Copy link
Member

fjy commented Mar 21, 2018

👍 This is really helpful

@b-slim
Copy link
Contributor

b-slim commented Mar 21, 2018

@AlexanderSaydakov thanks!

@b-slim b-slim merged commit 28920b1 into druid-io:src Mar 21, 2018
@fjy
Copy link
Member

fjy commented Mar 21, 2018

@AlexanderSaydakov all our docs are actually on https://github.com/druid-io/druid

Do you mind submitting there again?

gianm added a commit to gianm/druid-io.github.io that referenced this pull request Mar 21, 2018
fjy pushed a commit that referenced this pull request Mar 21, 2018
@gianm
Copy link
Member

gianm commented Mar 21, 2018

Reversion patch in #449. Once the docs are in the main druid-io/druid repo then the release scripts will copy them here, upon next release.

@gianm
Copy link
Member

gianm commented Mar 21, 2018

I'm working on a patch porting these to the Druid repo.

gianm added a commit to gianm/druid that referenced this pull request Mar 21, 2018
Originally written by @AlexanderSaydakov in druid-io/druid-io.github.io#448.
I also added redirects and updated links to point to the new
datasketches-extension.html landing page for the extension, rather than to
the old page about theta sketches.
@gianm
Copy link
Member

gianm commented Mar 21, 2018

Main Druid repo patch in: apache/druid#5513

Thanks @AlexanderSaydakov!

@AlexanderSaydakov AlexanderSaydakov deleted the datasketches-docs branch March 21, 2018 23:29
gianm added a commit to apache/druid that referenced this pull request Mar 22, 2018
Originally written by @AlexanderSaydakov in druid-io/druid-io.github.io#448.
I also added redirects and updated links to point to the new
datasketches-extension.html landing page for the extension, rather than to
the old page about theta sketches.
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