techdocsmith commented on code in PR #13090:
URL: https://github.com/apache/druid/pull/13090#discussion_r972204620


##########
docs/multi-stage-query/msq-api.md:
##########
@@ -57,7 +57,7 @@ POST /druid/v2/sql/task
 
 ```json
 {
-  "query": "INSERT INTO wikipedia\nSELECT\n  TIME_PARSE(\"timestamp\") AS 
__time,\n  *\nFROM TABLE(\n  EXTERN(\n    '{\"type\": \"http\", \"uris\": 
[\"https://static.imply.io/data/wikipedia.json.gz\"]}',\n    '{\"type\": 
\"json\"}',\n    '[{\"name\": \"added\", \"type\": \"long\"}, {\"name\": 
\"channel\", \"type\": \"string\"}, {\"name\": \"cityName\", \"type\": 
\"string\"}, {\"name\": \"comment\", \"type\": \"string\"}, {\"name\": 
\"commentLength\", \"type\": \"long\"}, {\"name\": \"countryIsoCode\", 
\"type\": \"string\"}, {\"name\": \"countryName\", \"type\": \"string\"}, 
{\"name\": \"deleted\", \"type\": \"long\"}, {\"name\": \"delta\", \"type\": 
\"long\"}, {\"name\": \"deltaBucket\", \"type\": \"string\"}, {\"name\": 
\"diffUrl\", \"type\": \"string\"}, {\"name\": \"flags\", \"type\": 
\"string\"}, {\"name\": \"isAnonymous\", \"type\": \"string\"}, {\"name\": 
\"isMinor\", \"type\": \"string\"}, {\"name\": \"isNew\", \"type\": 
\"string\"}, {\"name\": \"isRobot\", \"type\": \"string
 \"}, {\"name\": \"isUnpatrolled\", \"type\": \"string\"}, {\"name\": 
\"metroCode\", \"type\": \"string\"}, {\"name\": \"namespace\", \"type\": 
\"string\"}, {\"name\": \"page\", \"type\": \"string\"}, {\"name\": 
\"regionIsoCode\", \"type\": \"string\"}, {\"name\": \"regionName\", \"type\": 
\"string\"}, {\"name\": \"timestamp\", \"type\": \"string\"}, {\"name\": 
\"user\", \"type\": \"string\"}]'\n  )\n)\nPARTITIONED BY DAY",
+  "query": "INSERT INTO wikipedia\nSELECT\n  TIME_PARSE(\"timestamp\") AS 
__time,\n  *\nFROM TABLE(\n  EXTERN(\n    '{\"type\": \"http\", \"uris\": 
[\"https://druid.apache.org/data/wikipedia.json.gz\"]}',\n    '{\"type\": 
\"json\"}',\n    '[{\"name\": \"added\", \"type\": \"long\"}, {\"name\": 
\"channel\", \"type\": \"string\"}, {\"name\": \"cityName\", \"type\": 
\"string\"}, {\"name\": \"comment\", \"type\": \"string\"}, {\"name\": 
\"commentLength\", \"type\": \"long\"}, {\"name\": \"countryIsoCode\", 
\"type\": \"string\"}, {\"name\": \"countryName\", \"type\": \"string\"}, 
{\"name\": \"deleted\", \"type\": \"long\"}, {\"name\": \"delta\", \"type\": 
\"long\"}, {\"name\": \"deltaBucket\", \"type\": \"string\"}, {\"name\": 
\"diffUrl\", \"type\": \"string\"}, {\"name\": \"flags\", \"type\": 
\"string\"}, {\"name\": \"isAnonymous\", \"type\": \"string\"}, {\"name\": 
\"isMinor\", \"type\": \"string\"}, {\"name\": \"isNew\", \"type\": 
\"string\"}, {\"name\": \"isRobot\", \"type\": \"strin
 g\"}, {\"name\": \"isUnpatrolled\", \"type\": \"string\"}, {\"name\": 
\"metroCode\", \"type\": \"string\"}, {\"name\": \"namespace\", \"type\": 
\"string\"}, {\"name\": \"page\", \"type\": \"string\"}, {\"name\": 
\"regionIsoCode\", \"type\": \"string\"}, {\"name\": \"regionName\", \"type\": 
\"string\"}, {\"name\": \"timestamp\", \"type\": \"string\"}, {\"name\": 
\"user\", \"type\": \"string\"}]'\n  )\n)\nPARTITIONED BY DAY",

Review Comment:
   nit: Is it possible to format this query so that you don't have to scroll 
far to the right to see the whole query?



##########
docs/multi-stage-query/msq-api.md:
##########
@@ -652,25 +338,25 @@ print(response.text)
 
 ```
 {
-    "task": "query-a6b65442-f77e-44e4-af28-ab3b711a27ac",
-    "status": {
-        "id": "query-a6b65442-f77e-44e4-af28-ab3b711a27ac",
-        "groupId": "query-a6b65442-f77e-44e4-af28-ab3b711a27ac",
-        "type": "query_controller",
-        "createdTime": "2022-07-27T20:09:23.551Z",
-        "queueInsertionTime": "1970-01-01T00:00:00.000Z",
-        "statusCode": "SUCCESS",
-        "status": "SUCCESS",
-        "runnerStatusCode": "WAITING",
-        "duration": 136636,
-        "location": {
-            "host": "ip-10-201-5-81.ec2.internal",
-            "port": -1,
-            "tlsPort": 8100
-        },
-        "dataSource": "kttm_simple",
-        "errorMsg": null
-    }
+  "task": "query-3dc0c45d-34d7-4b15-86c9-cdb2d3ebfc4e",
+  "status": {
+    "id": "query-3dc0c45d-34d7-4b15-86c9-cdb2d3ebfc4e",
+    "groupId": "query-3dc0c45d-34d7-4b15-86c9-cdb2d3ebfc4e",
+    "type": "query_controller",
+    "createdTime": "2022-09-14T22:12:00.183Z",
+    "queueInsertionTime": "1970-01-01T00:00:00.000Z",
+    "statusCode": "RUNNING",
+    "status": "RUNNING",
+    "runnerStatusCode": "RUNNING",
+    "duration": -1,
+    "location": {
+      "host": "localhost",
+      "port": 8100,
+      "tlsPort": -1
+    },
+    "dataSource": "kttm_simple",
+    "errorMsg": null
+  }
 }
 ```
 

Review Comment:
   should line 382 / 389 be
   ```
   GET /druid/indexer/v1/task/<taskId>/reports
   ```
   Python example uses reports. Report singular 404 for me



##########
docs/multi-stage-query/msq-api.md:
##########
@@ -72,7 +72,7 @@ Make sure you replace `username`, `password`, 
`your-instance`, and `port` with t
 curl --location --request POST 
'https://<username>:<password>@<your-instance>:<port>/druid/v2/sql/task/' \
 --header 'Content-Type: application/json' \
 --data-raw '{
-    "query": "INSERT INTO wikipedia\nSELECT\n  TIME_PARSE(\"timestamp\") AS 
__time,\n  *\nFROM TABLE(\n  EXTERN(\n    '\''{\"type\": \"http\", \"uris\": 
[\"https://static.imply.io/data/wikipedia.json.gz\"]}'\'',\n    '\''{\"type\": 
\"json\"}'\'',\n    '\''[{\"name\": \"added\", \"type\": \"long\"}, {\"name\": 
\"channel\", \"type\": \"string\"}, {\"name\": \"cityName\", \"type\": 
\"string\"}, {\"name\": \"comment\", \"type\": \"string\"}, {\"name\": 
\"commentLength\", \"type\": \"long\"}, {\"name\": \"countryIsoCode\", 
\"type\": \"string\"}, {\"name\": \"countryName\", \"type\": \"string\"}, 
{\"name\": \"deleted\", \"type\": \"long\"}, {\"name\": \"delta\", \"type\": 
\"long\"}, {\"name\": \"deltaBucket\", \"type\": \"string\"}, {\"name\": 
\"diffUrl\", \"type\": \"string\"}, {\"name\": \"flags\", \"type\": 
\"string\"}, {\"name\": \"isAnonymous\", \"type\": \"string\"}, {\"name\": 
\"isMinor\", \"type\": \"string\"}, {\"name\": \"isNew\", \"type\": 
\"string\"}, {\"name\": \"isRobot\", \
 "type\": \"string\"}, {\"name\": \"isUnpatrolled\", \"type\": \"string\"}, 
{\"name\": \"metroCode\", \"type\": \"string\"}, {\"name\": \"namespace\", 
\"type\": \"string\"}, {\"name\": \"page\", \"type\": \"string\"}, {\"name\": 
\"regionIsoCode\", \"type\": \"string\"}, {\"name\": \"regionName\", \"type\": 
\"string\"}, {\"name\": \"timestamp\", \"type\": \"string\"}, {\"name\": 
\"user\", \"type\": \"string\"}]'\''\n  )\n)\nPARTITIONED BY DAY",
+    "query": "INSERT INTO wikipedia\nSELECT\n  TIME_PARSE(\"timestamp\") AS 
__time,\n  *\nFROM TABLE(\n  EXTERN(\n    '\''{\"type\": \"http\", \"uris\": 
[\"https://druid.apache.org/data/wikipedia.json.gz\"]}'\'',\n    '\''{\"type\": 
\"json\"}'\'',\n    '\''[{\"name\": \"added\", \"type\": \"long\"}, {\"name\": 
\"channel\", \"type\": \"string\"}, {\"name\": \"cityName\", \"type\": 
\"string\"}, {\"name\": \"comment\", \"type\": \"string\"}, {\"name\": 
\"commentLength\", \"type\": \"long\"}, {\"name\": \"countryIsoCode\", 
\"type\": \"string\"}, {\"name\": \"countryName\", \"type\": \"string\"}, 
{\"name\": \"deleted\", \"type\": \"long\"}, {\"name\": \"delta\", \"type\": 
\"long\"}, {\"name\": \"deltaBucket\", \"type\": \"string\"}, {\"name\": 
\"diffUrl\", \"type\": \"string\"}, {\"name\": \"flags\", \"type\": 
\"string\"}, {\"name\": \"isAnonymous\", \"type\": \"string\"}, {\"name\": 
\"isMinor\", \"type\": \"string\"}, {\"name\": \"isNew\", \"type\": 
\"string\"}, {\"name\": \"isRobot\", 
 \"type\": \"string\"}, {\"name\": \"isUnpatrolled\", \"type\": \"string\"}, 
{\"name\": \"metroCode\", \"type\": \"string\"}, {\"name\": \"namespace\", 
\"type\": \"string\"}, {\"name\": \"page\", \"type\": \"string\"}, {\"name\": 
\"regionIsoCode\", \"type\": \"string\"}, {\"name\": \"regionName\", \"type\": 
\"string\"}, {\"name\": \"timestamp\", \"type\": \"string\"}, {\"name\": 
\"user\", \"type\": \"string\"}]'\''\n  )\n)\nPARTITIONED BY DAY",
     "context": {
         "maxNumTasks": 3
     }

Review Comment:
   are we missing a closing \` for `data-raw`?



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