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


##########
docs/tutorials/tutorial-delete-data.md:
##########
@@ -27,7 +27,8 @@ sidebar_label: "Deleting data"
 This tutorial demonstrates how to delete existing data.
 
 For this tutorial, we'll assume you've already downloaded Apache Druid as 
described in
-the [single-machine quickstart](index.md) and have it running on your local 
machine.
+the [single-machine quickstart](index.md) and have it running on your local 
machine. In addition,
+we'll assume you've [jq](https://stedolan.github.io/jq/download/) installed in 
your machine.

Review Comment:
   Suggested rephrasing:
   
   ```
   This tutorial requires the following:
   * A running Apache Druid instance. If you don't have Druid, see the 
[single-machine quickstart](index.md) to get started.
   * The command-line JSON processor, 
[jq](https://stedolan.github.io/jq/download/).
   ```



##########
docs/tutorials/tutorial-delete-data.md:
##########
@@ -97,35 +97,29 @@ $ ls -l1 var/druid/segments/deletion-tutorial/
 
 Let's disable some segments by their segmentID. This will again mark the 
segments as "unused", but not remove them from deep storage. You can see the 
full segmentID for a segment from UI as explained below.
 
-In the [segments view](http://localhost:8888/unified-console.html#segments), 
click the arrow on the left side of one of the remaining segments to expand the 
segment entry:
-
-![Segments](../assets/tutorial-deletion-01.png "Segments")
+In the [segments view](http://localhost:8888/unified-console.html#segments), 
click one of the segment rows to open the segment metadata dialog:
 
-The top of the info box shows the full segment ID, e.g. 
`deletion-tutorial_2015-09-12T14:00:00.000Z_2015-09-12T15:00:00.000Z_2019-02-28T01:11:51.606Z`
 for the segment of hour 14.
+![Segments_2](../assets/tutorial-deletion-02.png "Segments 2")
 
-Let's disable the hour 13 and 14 segments by sending a POST request to the 
Coordinator with this payload
+The `identifier` in the metadata dialog shows the full segment ID, e.g. 
`deletion-tutorial_2015-09-12T23:00:00.000Z_2015-09-13T00:00:00.000Z_2023-05-16T00:04:12.091Z`
 for the segment of hour 23.

Review Comment:
   ```suggestion
   The `identifier` field in the metadata dialog shows the full segment ID. For 
example, the hour 23 segment has segment ID 
`deletion-tutorial_2015-09-12T23:00:00.000Z_2015-09-13T00:00:00.000Z_2023-05-16T00:04:12.091Z`.
   ```



##########
docs/tutorials/tutorial-delete-data.md:
##########
@@ -97,35 +97,29 @@ $ ls -l1 var/druid/segments/deletion-tutorial/
 
 Let's disable some segments by their segmentID. This will again mark the 
segments as "unused", but not remove them from deep storage. You can see the 
full segmentID for a segment from UI as explained below.

Review Comment:
   ```suggestion
   Let's disable some segments by their segmentID. This will again mark the 
segments as "unused", but not remove them from deep storage. You can see the 
full segmentID for a segment using the web console.
   ```



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


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

Reply via email to