pengding-stripe opened a new issue, #17329: URL: https://github.com/apache/pinot/issues/17329
Pinot today returns 200 http response code even though the query failed, it has an exceptions field with detailed errors + Pinot error code. This design doesn't work well with some cases that depend on the http response code returned from Pinot broker. There was another [ask](https://github.com/apache/pinot/issues/12190) for adding a response header to reflect the Pinot error code but that doesn't solve the problem when there's need depending on the http response code only to tell if the query succeeds or not. To make the changes backward-compatible, I think we can introduce a request header like `X-PINOT-STRICT-ERROR` and Pinot should return a 4xx or 5xx depending on the Pinot error code if it's set to true. This way users need this feature can configure the upstream accordingly without affecting use cases that do not care about the http response code. -- 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]
