Re: code refactoring suggestion

2017-03-03 Thread Andrey Gura
Aleksey, once again, listener will be called directly in GridFutureAdapter.listen() method if future already completed. 1. execute prepareAsync() 2. prepare phase answer got 3. listener binded (actually not binded but called) @Override public void listen(IgniteInClosure> lsnr0) { assert lsnr

Re: code refactoring suggestion

2017-03-02 Thread voipp
Hi all ! During pessimistic transaction we execute prepare phase at org.apache.ignite.internal.processors.cache.distributed.near.GridNearTxLocal#commitAsync after executing prepareAsync(); We bind listener on prepare future. Perhaps its better initially bind listener and then execute fut.prepa

Re: code refactoring suggestion

2017-03-02 Thread Andrey Gura
Aleksey, if you talk about fut.listen() then it doesn't make sense. listen() call checks whether future is already completed and if it completed invokes passed listener from current thread. On Thu, Mar 2, 2017 at 12:42 PM, ALEKSEY KUZNETSOV wrote: > Hi all ! During pessimistic transaction we exe