ignite-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Pavel Kovalenko <jokse...@gmail.com>
Subject Re: [DISCUSSION] Remove preload predicate in GridCachePreloader
Date Fri, 19 Jul 2019 13:07:43 GMT
Maxim,

Go ahead. I don't see any real usages of this functionality and seems it
obsolete and should be removed.

пт, 19 июл. 2019 г. в 08:36, Anton Vinogradov <av@apache.org>:

> Folks,
>
> To be clear, I'm going to merge the PR next Monday in case of interest lack
> (which means the feature is obsolete and can be removed).
>
> On Thu, Jul 18, 2019 at 2:29 PM Maxim Muzafarov <maxmuzaf@gmail.com>
> wrote:
>
> > Igniters,
> >
> > It seems to me that GridCachePreloader.preloadPredicate(); currently
> > not used and even not tested at all and can be removed. Can anyone
> > confirm that?
> >
> > I see the following disadvantages here:
> > - not used in project code, not tested;
> > - mvcc preload entry doesn't use this filter (regular preload entry
> does);
> > - a bit strange removing any entries on rebalancing under the hood, it
> > can lead to data inconsistency
> >
> > I've created the issue [1] and PR [2] with removing such methods (bot
> > visa green).
> > WDYT?
> >
> > [1] https://issues.apache.org/jira/browse/IGNITE-11989
> > [2] https://github.com/apache/ignite/pull/6699/files
> >
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message