[GitHub] [druid] sthetland commented on a change in pull request #9879: Querying doc refresh tutorial

[GitBox <gi...@apache.org>]

sthetland commented on a change in pull request #9879:
URL: https://github.com/apache/druid/pull/9879#discussion_r429477664



##########
File path: docs/querying/querying.md
##########
@@ -44,7 +49,7 @@ The Content-Type/Accept Headers can also take 'application/x-jackson-smile'.
 curl -X POST '<queryable_host>:<port>/druid/v2/?pretty' -H 'Content-Type:application/json' -H 'Accept:application/x-jackson-smile' -d @<query_json_file>
 ```

Review comment:
       Thank you for review!... About: 
   
   >Can you double check the default ports used in the overall tutorial. Some places it uses 8888, but druid broker default seems to be 8082.
   
   After trying them out with Druid in the quickstart config, both ports work, which makes sense, since it looks like the Broker is 8082 and the Router is at 8888. Properly though,  such requests should come through the router, no? 




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

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



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