ignite-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alexander Menshikov (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (IGNITE-8714) CacheEntryEvent.getOldValue should be available
Date Wed, 27 Jun 2018 17:08:00 GMT

    [ https://issues.apache.org/jira/browse/IGNITE-8714?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16525301#comment-16525301

Alexander Menshikov commented on IGNITE-8714:

Unfortunately, I will be too busy soon and I'm not gonna do the task in close time. My current
PR have problems with some .NET tests. Also, it is a bit dirty. But still, I think it's easier
to fix my PR then start work from a scratch.

[~NSAmelchev] can you finish the work?

> CacheEntryEvent.getOldValue should be available
> -----------------------------------------------
>                 Key: IGNITE-8714
>                 URL: https://issues.apache.org/jira/browse/IGNITE-8714
>             Project: Ignite
>          Issue Type: Sub-task
>            Reporter: Alexander Menshikov
>            Assignee: Anton Vinogradov
>            Priority: Major
>              Labels: iep-21
>             Fix For: 2.7
> A table is copied from speck changing discussion:
> When CacheEntryListenerConfiguration#isOldValueRequired is *true*:
> ||CacheEntryEvent Method||CreatedListener||UpdatedListener||Removed/Expired Listener||
> |getValue()|_value_|_value_|_oldValue_|
> |getOldValue()|null|_oldValue_|_oldValue_|
> The last column is new behavior for #getValue().
> Сurrent master fails with this reason on following tests:
>  * CacheListenerTest.testFilteredListener
>  * CacheListenerTest.testCacheEntryListener
> I have to say tests testFilteredListener and testCacheEntryListener also fail in `afterTests`
section because of IGNITE-8715, so PR will not change the number of failed tests if ignite-8715
unfixed. But it will change the reason – please see the log.
> Please see also link on JCache TCK and speck 1.1.0 changes.
> *UPD:*
> There are a lot of our tests wich going to break by this change. Also, we use events
in Services deployment, which will be affected too.

This message was sent by Atlassian JIRA

View raw message