-
Notifications
You must be signed in to change notification settings - Fork 156
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Missing null query check in query response #137
Comments
this is likely something https://trino.io/blog/2021/01/04/migrating-from-prestosql-to-trino.html has described |
We may consider adding dual HTTP header writes (if either |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
For an incompatible client, Trino/Presto throw exception :
and sends no Query Id information.
We are currently submitting the query for addition to Query History without checking if the Query Id is
null
.presto-gateway/gateway-ha/src/main/java/com/lyft/data/gateway/ha/handler/QueryIdCachingProxyHandler.java
Line 258 in 39d7d5f
which leads to an Exception
The text was updated successfully, but these errors were encountered: