eaugene opened a new pull request, #13292:
URL: https://github.com/apache/pinot/pull/13292

   The message from WebApplicationException Permission Denied is not sent back 
as HTTP response from broker. This minor patch fixes that. 
   
   **Existing**
   ```
   $ curl -X POST "http://<host:port>/query/sql"      -H "Content-Type: 
application/json"      -d '{"sql":"<QUERY>", "queryOptions": 
"useMultistageEngine=true"}' -v
   Note: Unnecessary use of -X or --request, POST is already inferred.
   *   Trying <IP>...
   * TCP_NODELAY set
   * Connected to <HOST> (<IP>) port <port> (#0)
   > POST /query/sql HTTP/1.1
   > Host: <host:port>
   > User-Agent: curl/7.64.0
   > Accept: */*
   > Content-Type: application/json
   > Content-Length: 158
   > 
   * upload completely sent off: 158 out of 158 bytes
   < HTTP/1.1 403 Forbidden
   < Content-Length: 0
   < 
   * Connection #0 to host <host> left intact
   ```
   
   **With this Change**
   ```
   $ curl -X POST "http://<host:port>/query/sql"      -H "Content-Type: 
application/json"      -d '{"sql":"<QUERY>", "queryOptions": 
"useMultistageEngine=true"}' -v
   Note: Unnecessary use of -X or --request, POST is already inferred.
   *   Trying <IP>...
   * TCP_NODELAY set
   * Connected to <HOST> (<IP>) port <port> (#0)
   > POST /query/sql HTTP/1.1
   > Host: <host:port>
   > User-Agent: curl/7.64.0
   > Accept: */*
   > Content-Type: application/json
   > Content-Length: 158
   > 
   * upload completely sent off: 158 out of 158 bytes
   < HTTP/1.1 403 Forbidden
   < Content-Length: 0
   < 
   * Connection #0 to host <host> left intact
   Permission denied.
   ```
   
   This would be useful when the failed table names are returned as from the 
change made in https://github.com/apache/pinot/pull/13195


-- 
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...@pinot.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org

Reply via email to