gianm commented on a change in pull request #6916: Add master/data/query server 
concepts to docs/packaging
URL: https://github.com/apache/incubator-druid/pull/6916#discussion_r252028071
 
 

 ##########
 File path: docs/content/comparisons/druid-vs-redshift.md
 ##########
 @@ -42,7 +42,7 @@ Druid’s write semantics are not as fluid and does not 
support full joins (we s
 
 ### Data distribution model
 
-Druid’s data distribution is segment-based and leverages a highly available 
"deep" storage such as S3 or HDFS. Scaling up (or down) does not require 
massive copy actions or downtime; in fact, losing any number of historical 
nodes does not result in data loss because new historical nodes can always be 
brought up by reading data from "deep" storage.
+Druid’s data distribution is segment-based and leverages a highly available 
"deep" storage such as S3 or HDFS. Scaling up (or down) does not require 
massive copy actions or downtime; in fact, losing any number of Historical 
nodes does not result in data loss because new Historical nodes can always be 
brought up by reading data from "deep" storage.
 
 Review comment:
   I think it would help but would rather do it in a different PR.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to