tagomoris / presto-client-node

Distributed query engine Presto client library for node.js
MIT License
126 stars 57 forks source link

Retrieving query status and cancelling query #75

Open bblodg opened 9 months ago

bblodg commented 9 months ago

It seems that the kill and query commands call the "v1/query/" endpoint, however I'm getting the following 404 path not found error: {"timestamp: "...", "status":404, "error": "Not Found", "path":"v1/query/{query-id}"}

I noticed that in Presto + Trino documentation, they say that these get and delete requests should be made to the nextUri, which include the "v1/statement" path as opposed to above: https://prestodb.io/docs/current/develop/client-protocol.html

MasterOdin commented 7 months ago

Looks like doing DELETE /v1/query/{query-id} was the suggested mechanism back in 2019 (https://github.com/prestodb/presto/wiki/HTTP-Protocol), but they've since changed that at some point to instead doing the DELETE against the nextUri.