Hello, Andrey.

Thanks for positive feedback.
Appreciate it.

> we can't cancel service or service's method

I understand it. 
Seems, we have several obvious options here:

* Try to do it and if fail answer to the user: «I’ve tried and fail. Sorry»
* Kill hanging thread.

> This invocations not register anywhere and can't be tracked and killed.

Yes.
So we should invent tracking for those invocation to be able to kill it :)


> 17 янв. 2020 г., в 21:03, Andrey Gura <ag...@apache.org> написал(а):
> 
> It would be grate.
> 
> Only one comment: we can't cancel service or service's method
> execution because service grid API doesn't imply any requirement to
> service implementation. So if user do something like infinite cycle or
> blocking but non-interruptible operation it's impossible to interrupt
> it. Also service method invocation itself isn't cluster wide or local
> node specific operation. This invocations not register anywhere and
> can't be tracked and killed.
> 
> Unfortunately, the same is valid for some jobs in sense of infinite or
> non-interruptible operations.
> 
> On Wed, Jan 15, 2020 at 9:30 PM Николай Ижиков <nizhi...@apache.org> wrote:
>> 
>> Hello, Igniters.
>> 
>> As you may know, we put a lot of effort to improve Ignite metric and 
>> diagnostic API.
>> We have created the following API:
>>    * Metric manager
>>    * System view manager
>> As far as I know, we would have tracing capabilities soon.
>> 
>> I think it's time to take the next step.
>> We should provide to the administrator the API to cancel(stop, kill) 
>> arbitrary user started computation.
>> 
>> Right now we have API to do it for:
>>    * transactions `TransactionsMXBean#getActiveTransactions`.
>>    * SQL queries: `KILL QUERY` sql command and visor task - 
>> `VisorQueryCancelTask`
>> 
>> Please, note, these features are implemented via different API.
>> 
>> I think we should introduce uniform Cancel API for the following 
>> computations:
>> 
>>    * service.
>>    * specific service method execution.
>>    * compute job(compute task).
>>    * query(scan, continuous, text).
>> 
>> We should  also rework kill transaction and kill SQL queries API to make 
>> them similar to each other.
>> I have plans to write an IEP for it and implement it.
>> What do you think?
>> 

Reply via email to