Conversation
Cool product
jihoonson
left a comment
jihoonson
left a comment
There was a problem hiding this comment.
Hi @tlblessing, thank you for your contribution! It looks good except the spell error which our spell check bot noticed 🙂. Would you please fix it?
../docs/design/index.md
57 | ruid servers fail, the system automaticallys route around the damage until
>> 1 spelling error found in 167 files
|
|
||
| - 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 |
There was a problem hiding this comment.
Actually "group by" is the name of one of our query types. The doc mostly uses "groupBy". How about using the same notation here too?
|
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 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. |
2 participants
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.