zhaoyongjie commented on pull request #13868: URL: https://github.com/apache/superset/pull/13868#issuecomment-810427037
> Is 422 correct here? I think we made it 500 originally because we don't know if the backend failed or the client. 422 also means "the syntax of the request entity is correct, but it was unable to process the contained instructions", which seems to indicate that it's not the right error for a SQL query with bad syntax. If we definitely want to make this a client error, perhaps `400` is the safest option? Maybe we need a set of response status code set. it is not necessary for all status codes to respond. -- 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: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
