You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@druid.apache.org by GitBox <gi...@apache.org> on 2020/09/14 13:49:40 UTC
[GitHub] [druid] tlblessing opened a new pull request #10396: Update index.md
tlblessing opened a new pull request #10396:
URL: https://github.com/apache/druid/pull/10396
Cool product.
Minor suggested copy edits.
-avoid excessive use of double quotes for terms that are well understood.
-avoid future tense when possible.
-missing punctuation.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org
[GitHub] [druid] stale[bot] commented on pull request #10396: Update index.md
Posted by GitBox <gi...@apache.org>.
stale[bot] commented on pull request #10396:
URL: https://github.com/apache/druid/pull/10396#issuecomment-727716083
This pull request has been marked as stale due to 60 days of inactivity. It will be closed in 4 weeks if no further activity occurs. If you think that's incorrect or this pull request should instead be reviewed, please simply write any comment. Even if closed, you can still revive the PR at any time or discuss it on the dev@druid.apache.org list. Thank you for your contributions.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org
[GitHub] [druid] stale[bot] commented on pull request #10396: Update index.md
Posted by GitBox <gi...@apache.org>.
stale[bot] commented on pull request #10396:
URL: https://github.com/apache/druid/pull/10396#issuecomment-751238566
This pull request/issue has been closed due to lack of activity. If you think that is incorrect, or the pull request requires review, you can revive the PR at any time.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org
[GitHub] [druid] jihoonson commented on a change in pull request #10396: Update index.md
Posted by GitBox <gi...@apache.org>.
jihoonson commented on a change in pull request #10396:
URL: https://github.com/apache/druid/pull/10396#discussion_r489076874
##########
File path: docs/design/index.md
##########
@@ -77,24 +77,24 @@ summarization partially pre-aggregates your data, and can lead to big costs savi
## When should I use Druid?
Druid is used by many companies of various sizes for many different use cases. Check out the
-[Powered by Apache Druid](/druid-powered) page
+[Powered by Apache Druid](/druid-powered) page.
Druid is likely a good choice if your use case fits a few of the following descriptors:
- Insert rates are very high, but updates are less common.
-- Most of your queries are aggregation and reporting queries ("group by" queries). You may also have searching and
+- Most of your queries are aggregation and reporting queries (group by queries). You may also have searching and
Review comment:
Actually "group by" is the name of one of our query types. The [doc](https://druid.apache.org/docs/latest/querying/groupbyquery.html) mostly uses "groupBy". How about using the same notation here too?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org
[GitHub] [druid] stale[bot] closed pull request #10396: Update index.md
Posted by GitBox <gi...@apache.org>.
stale[bot] closed pull request #10396:
URL: https://github.com/apache/druid/pull/10396
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org