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