Re: cache store usability

2018-05-02 Thread Dmitriy Setrakyan
On Wed, May 2, 2018 at 4:15 PM, Valentin Kulichenko < valentin.kuliche...@gmail.com> wrote: > Vyacheslav, > > There is already a warning for this: > https://github.com/apache/ignite/blob/master/modules/ > core/src/main/java/org/apache/ignite/internal/processors/cache/store/ >

Re: cache store usability

2018-05-02 Thread Valentin Kulichenko
Vyacheslav, There is already a warning for this: https://github.com/apache/ignite/blob/master/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/store/GridCacheStoreManagerAdapter.java#L225 -Val On Wed, May 2, 2018 at 11:31 AM, Vyacheslav Daradur wrote:

Re: cache store usability

2018-05-02 Thread Vyacheslav Daradur
I agree I have noticed that too. But we can't just call setWriteThrough(true) if setWriteBehindEnabled(true) was turned on because in this case CacheStoryFactory should be properly set before. Maybe it will be better to print to the log a warning which will notify the user that

cache store usability

2018-05-02 Thread Dmitriy Setrakyan
Igniters, This is another usability issue that can be addressed quickly. Apparently, setWriteBehindEnabled(true) is not enough to enable CacheStore, the setWriteThrough(true) also needs to be enabled. https://stackoverflow.com/questions/50118842/write-behind-and-write-through Why not make it