Re: IGNITE-4535 : Add option to store deserialized values on-heap

2017-04-05 Thread Dmitriy Setrakyan
On Wed, Apr 5, 2017 at 2:52 AM, Ilya Lantukh wrote: > Dmitry, > > 1. Setting an eviction policy should not be a mechanism to enable the > > on-heap cache. We already have eviction policies off-heap as well, and > they > > don't enable anything. On top of that, the eviction policy should not be >

Re: IGNITE-4535 : Add option to store deserialized values on-heap

2017-04-05 Thread Sergi Vladykin
I think having pure on-heap cache is good idea. Sergi 2017-04-05 12:52 GMT+03:00 Ilya Lantukh : > Dmitry, > > 1. Setting an eviction policy should not be a mechanism to enable the > > on-heap cache. We already have eviction policies off-heap as well, and > they > > don't enable anything. On top

Re: IGNITE-4535 : Add option to store deserialized values on-heap

2017-04-05 Thread Ilya Lantukh
Dmitry, 1. Setting an eviction policy should not be a mechanism to enable the > on-heap cache. We already have eviction policies off-heap as well, and they > don't enable anything. On top of that, the eviction policy should not be a > requirement for the on-heap cache. User should still be able to

Re: IGNITE-4535 : Add option to store deserialized values on-heap

2017-04-04 Thread Dmitriy Setrakyan
Ilya, I looked at the Semyon's comments in the ticket, and I think I agree with him on all counts. 1. Setting an eviction policy should not be a mechanism to enable the on-heap cache. We already have eviction policies off-heap as well, and they don't enable anything. On top of that, the eviction p