ektravel commented on code in PR #16762:
URL: https://github.com/apache/druid/pull/16762#discussion_r1691834315


##########
docs/tutorials/tutorial-rollup.md:
##########
@@ -24,18 +24,21 @@ sidebar_label: Aggregate data with rollup
   -->
 
 
-Apache Druid can summarize raw data at ingestion time using a process we refer 
to as "rollup". Rollup is a first-level aggregation operation over a selected 
set of columns that reduces the size of stored data.
+Apache Druid® can summarize raw data at ingestion time using a 
process known as "rollup." [Rollup](../ingestion/rollup.md) is a first-level 
aggregation operation over a selected set of columns that reduces the size of 
stored data.
 
-This tutorial will demonstrate the effects of rollup on an example dataset.
+The tutorial demonstrates how to apply rollup at ingestion and shows the 
effect of rollup at query time. See [ingesting with 
rollup](https://druid.apache.org/docs/latest/multi-stage-query/concepts/#rollup)
 to learn more. 

Review Comment:
   ```suggestion
   This tutorial demonstrates how to apply rollup during ingestion and 
highlights its effects during query execution. 
   ```
   Remove this line as it doesn't apply to the preceding sentence. Replace the 
link on line 27 with this link 
(https://druid.apache.org/docs/latest/multi-stage-query/concepts/#rollup) but 
use the relative path. 
   



-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org
For additional commands, e-mail: commits-h...@druid.apache.org

Reply via email to