-
Notifications
You must be signed in to change notification settings - Fork 569
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
ClickHouse QueryID Issue #870
Comments
Perhaps you can call Statement.cancel() so that you don't have to know the query ID and kill the query manually? |
emm... we don't use native jdbc(mybatis instead) , so it will be a little trouble to get statement . I mean that do we can find some ways to use my query ID not yours or give me some method to get your queryID so that I can kill query as I want ? : ) |
I see. Just did a quick search and it seems mybatis still does not support cancel. Anyway, I think we need to enhance the driver to expose methods for setting/getting query id. Before that, I can only think of AOP to replace query id passed to Clickhouse request.query(sql, queryId). |
good idea, thank you so much : ) |
I think there are a few issues requesting an interceptor and the ability to customize query id etc., I'll enhance the driver to expose an interface for this. |
Starting from 0.3.2-patch11, you should be able to customize query ID and session ID by implementing ClickHouseRequestManager. |
@ltl0814 Hello, I have a requirement similar to yours. I would like to ask how you use the query ID to cancel a query, and how effective it is in a distributed ClickHouse setup? |
sql : kill query where query_id = xxx on cluster xxx |
Hi , I have a scenario here , we need to get query ID before CH return result , because user don't know this query executed quickly or not , so we injected query ID by modifying the bytecode to override your queryID ,so that we can kill this query by accurate queryID, but these days we upgrade CH to 0.3.2 ,we found that you have changed this code by this way(below image) , and this is a bit tricky to override queryID again. so could you please advise this for us ? and I think modify bytecode is not a good way to inject query ID....

The text was updated successfully, but these errors were encountered: