Re: IGNITE-2795

2016-07-07 Thread Alexei Scherbakov
I've updated the fix. Currently all the copying is performed inside query iterators only if needed. 2016-07-07 14:38 GMT+03:00 Alexei Scherbakov : > Denis, > > I think the current state of fix for SQL queries is consistent with the > highlighted topics, except > >

Re: IGNITE-2795

2016-07-07 Thread Alexei Scherbakov
Denis, I think the current state of fix for SQL queries is consistent with the highlighted topics, except - server node mustn’t create a copy if a result set will be transferred to a remote node; I afraid it would be difficult to implement without interfering with current implementation too

Re: IGNITE-2795

2016-07-07 Thread Denis Magda
I would say that, yes, we should make copyOnRead flag consistent across all cache ‘get’ like operations. However we need to make sure that the copy is not created in the following cases - server node mustn’t create a copy if a result set will be transferred to a remote node; - client node or