On Wed, Apr 5, 2017 at 2:52 AM, Ilya Lantukh <ilant...@gridgain.com> 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 > a > > requirement for the on-heap cache. User should still be able to enable > the > > on-heap cache, even if it grows indefinitely without evictions. We should > > have a more intuitive flag here. > > > It doesn't make any sence to me to enable on-heap cache without evictions: > it will result in having all data both in on-heap and off-heap memory. If > we want to support such use case, we should implement a separate on-heap > only mode with page memory disabled. > I agree that this would cause a double memory consumption, but yet again, we don't have a pure on-heap cache yet. Until we do, we should not require an eviction policy present for the on-heap caches.
