[GitHub] asdf2014 commented on a change in pull request #7112: Added friendlier dsql error message for 405 (which occurs when druid.sql.enabled=false)

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

asdf2014 commented on a change in pull request #7112: Added friendlier dsql error message for 405 (which occurs when druid.sql.enabled=false)
URL: https://github.com/apache/incubator-druid/pull/7112#discussion_r261035285
 
 

 ##########
 File path: examples/bin/dsql-main
 ##########
 @@ -136,6 +136,10 @@ def raise_friendly_error(e):
       if error_obj['host']:
         error_text = error_text + ' (' + str(error_obj['host']) + ')'
       raise DruidSqlException(error_text)
+    elif e.code == 405:
+      error_text = 'HTTP Error {0}: {1}\n{2}'.format(e.code, e.reason + " - Are you using the correct broker URL and " +\
 
 Review comment:
   The backslash is redundant here.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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


With regards,
Apache Git Services

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