CacheLoadOnlyStoreAdapter

[Valentin Kulichenko]

Igniters,

We have a class that is called CacheLoadOnlyStoreAdapter. This is a cache
store adapter that implements utility logic for multithreaded data loading,
so that user has to provide only the logic that queries DB and parses DB
row into object. Everything else is done automatically.

It's a cool feature, but the name looks really confusing to me. It's
definitely not 'load-only' store, because it implements CacheWriter.

How about we rename it to CacheMultithreadedLoadStoreAdapter? Any other
suggestions?

-Val

[Vladimir Ozerov]

+1
I was really stuck when sew current name for the first time.

On Sat, Dec 12, 2015 at 3:32 AM, Valentin Kulichenko <
[hidden email]> wrote:

> Igniters,
>
> We have a class that is called CacheLoadOnlyStoreAdapter. This is a cache
> store adapter that implements utility logic for multithreaded data loading,
> so that user has to provide only the logic that queries DB and parses DB
> row into object. Everything else is done automatically.
>
> It's a cool feature, but the name looks really confusing to me. It's
> definitely not 'load-only' store, because it implements CacheWriter.
>
> How about we rename it to CacheMultithreadedLoadStoreAdapter? Any other
> suggestions?
>
> -Val
>

[yzhdanov]

Renaming will break config compatibility. I don't think it is a good idea
to rename this, but we can update documentation to make things better.

--Yakov

2015-12-13 19:25 GMT+03:00 Vladimir Ozerov <[hidden email]>:

> +1
> I was really stuck when sew current name for the first time.
>
> On Sat, Dec 12, 2015 at 3:32 AM, Valentin Kulichenko <
> [hidden email]> wrote:
>
> > Igniters,
> >
> > We have a class that is called CacheLoadOnlyStoreAdapter. This is a cache
> > store adapter that implements utility logic for multithreaded data
> loading,
> > so that user has to provide only the logic that queries DB and parses DB
> > row into object. Everything else is done automatically.
> >
> > It's a cool feature, but the name looks really confusing to me. It's
> > definitely not 'load-only' store, because it implements CacheWriter.
> >
> > How about we rename it to CacheMultithreadedLoadStoreAdapter? Any other
> > suggestions?
> >
> > -Val
> >
>