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


##########
docs/tutorials/tutorial-rollup.md:
##########
@@ -147,3 +152,17 @@ Therefore, no rollup takes place:
 | `2018-01-01T01:03:00.000Z` | `1.1.1.1` | `2.2.2.2` | `10,204` | `1` | `49` |
 
 
+## Learn More
+
+See the following topics for more information:
+
+* [SQL-based ingestion query 
examples](../multi-stage-query/examples.md/#insert-with-rollup) for another 
example of data rollup during ingestion.
+
+* [SQL-based ingestion concepts](../multi-stage-query/concepts/#rollup) for 
more details on the concept of rollup.
+
+* [Data rollup](../ingestion/rollup/) for suggestions and best practices when 
performing rollup.
+
+
+* [Druid schema model](../ingestion/schema-model.md) to go over more details 
on timestamp, dimensions, and metrics.
+
+

Review Comment:
   Make sure to use .md links (and verify that they work 🙂 )
   
   ```suggestion
   * [SQL-based ingestion query 
examples](../multi-stage-query/examples.md#insert-with-rollup) for another 
example of data rollup during ingestion.
   * [SQL-based ingestion concepts](../multi-stage-query/concepts.md#rollup) 
for more details on the concept of rollup.
   * [Data rollup](../ingestion/rollup.md) for suggestions and best practices 
when performing rollup.
   * [Druid schema model](../ingestion/schema-model.md) to go over more details 
on timestamp, dimensions, and metrics.
   
   ```



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