You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Just to clarify, currently the backend doesn't support parameterized queries with the query results API. The way it works, is that the UI merges the params with the query and sends the backend the resulting query. This is why #285 is still open.
The tricky part in implementing this, is that query execution is async and when calling for query result, you might get a job id back that you need to poll until you get results. I might add support for that and leave for the API caller to implement polling.
For parameters support over API, subscribe to #285.
I will keep this open, to fix the behavior you indicated (it should probably return some error).
In #1069 I fixed the wrong caching of semi-random results for parameterized queries and in #1116 I fixed the API call to return a meaningful error.
Closing this now. Support for parameters is already available in the POST /api/query_results API call and to know when GET /api/queries/<id>/results.json will support it track #285.
Repro steps:
create query with params, e.g. name:
select * from users where users.name= "{{name}}"
save, and attempt to retrieve the query like so:
/api/queries/{ID}/results.csv?p_name=foo&api_key=XXXXXXXX
Then, change to
p_name=bar
.Check results
Results: results for
foo
Expected: results for
bar
Version: 0.8.1.b1110
The text was updated successfully, but these errors were encountered: