Hi Alex, >> 1) Current implementation will collect info and cancel only "TwoStep" >> queries. >> Should we also support LOCAL queries? >>
I would make it transparent for the user giving ability to work with LOCAL queries as well. How difficult is to support this? >> 2) Should we also support SqlQuery / ScanQuery ? Isn’t the goal of this task to support all SQL queries? Don’t get why you ask about SqlQuery. As for ScanQuery I would do it as a next step. The review of the code should be done by Sergi. Let’s give him some time to do that. — Denis > On Jan 31, 2017, at 5:13 AM, Alexey Kuznetsov <akuznet...@apache.org> wrote: > > Any feedback, please? > > On Mon, Jan 30, 2017 at 9:38 PM, Alexey Kuznetsov <akuznet...@apache.org> > wrote: > >> Hi All! >> >> I prepared first version https://issues.apache.org/jira/browse/IGNITE-4436 >> (SQL: create API to collect list of running SQL queries and cancel them). >> >> Please review it and give a feedback: http://reviews.ignit >> e.apache.org/ignite/review/IGNT-CR-84 >> >> But also I have following questions: >> 1) Current implementation will collect info and cancel only "TwoStep" >> queries. >> Should we also support LOCAL queries? >> >> 2) Should we also support SqlQuery / ScanQuery ? >> >> > > > -- > Alexey Kuznetsov
