[IGNITE-8714] CacheEntryEvent.getOldValue should be available

• Type: Sub-task
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 2.7
• Labels: iep-21
• Assignee: Nikita Amelchev
• Reporter: Alexander Menshikov
• Created: 05/Jun/18 17:56
• Updated: 04/Sep/23 11:05
• Resolved: 02/Aug/18 13:10

Description

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.

Issue Links

• is related to: IGNITE-20327 [Thin clients] Continuous Query EXPIRY/REMOVE events can consumes a huge amount of heap
• relates to: IGNITE-9020 .NET: Creating CacheEntry events regardless of values.
• links to: GitHub Pull Request #4313
• links to: JCache 1.1 spec changes
• links to: JCache TCK 1.1 changes
• links to: TC tests
• links to: Upsource