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

Aleksey Plekhanov commented on IGNITE-13295:
--------------------------------------------

[~slava.koptilin], looks good to me. I've left one minor comment on github, 
it's almost not related to your changes, but as far as you refactor the test, I 
think it's better to remove redundant code in this test. But it's up to you, 
fix it or keep as is. 

> Expiration of cache entries, which relate to a non-persistent data region in 
> the persistent cluster, results in AssertionError
> ------------------------------------------------------------------------------------------------------------------------------
>
>                 Key: IGNITE-13295
>                 URL: https://issues.apache.org/jira/browse/IGNITE-13295
>             Project: Ignite
>          Issue Type: Bug
>    Affects Versions: 2.8.1
>            Reporter: Vyacheslav Koptilin
>            Assignee: Vyacheslav Koptilin
>            Priority: Major
>             Fix For: 2.10
>
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> In the case of the cluster uses AI native persistent and, at the same time, 
> it has a non-persistent data region and cache related to that non-persistent 
> region with expiry policy specified, the following Exception is thrown:
> {noformat}
> java.lang.AssertionError
>       at 
> org.apache.ignite.internal.processors.cache.IgniteCacheOffheapManagerImpl$CacheDataStoreImpl.remove(IgniteCacheOffheapManagerImpl.java:2678)
>       at 
> org.apache.ignite.internal.processors.cache.IgniteCacheOffheapManagerImpl.remove(IgniteCacheOffheapManagerImpl.java:619)
>       at 
> org.apache.ignite.internal.processors.cache.GridCacheMapEntry.removeValue(GridCacheMapEntry.java:4411)
>       at 
> org.apache.ignite.internal.processors.cache.GridCacheMapEntry.onExpired(GridCacheMapEntry.java:4103)
>       at 
> org.apache.ignite.internal.processors.cache.GridCacheMapEntry.onTtlExpired(GridCacheMapEntry.java:4028)
>       at 
> org.apache.ignite.internal.processors.cache.GridCacheTtlManager$1.applyx(GridCacheTtlManager.java:76)
>       at 
> org.apache.ignite.internal.processors.cache.GridCacheTtlManager$1.applyx(GridCacheTtlManager.java:67)
>       at 
> org.apache.ignite.internal.util.lang.IgniteInClosure2X.apply(IgniteInClosure2X.java:38)
>       at 
> org.apache.ignite.internal.processors.cache.IgniteCacheOffheapManagerImpl.expireInternal(IgniteCacheOffheapManagerImpl.java:1422)
>       at 
> org.apache.ignite.internal.processors.cache.IgniteCacheOffheapManagerImpl.expire(IgniteCacheOffheapManagerImpl.java:1367)
>       at 
> org.apache.ignite.internal.processors.cache.GridCacheTtlManager.expire(GridCacheTtlManager.java:243)
>       at 
> org.apache.ignite.internal.processors.cache.GridCacheSharedTtlCleanupManager$CleanupWorker.lambda$body$0(GridCacheSharedTtlCleanupManager.java:179)
>       at 
> java.util.concurrent.ConcurrentHashMap.computeIfPresent(ConcurrentHashMap.java:1769)
>       at 
> org.apache.ignite.internal.processors.cache.GridCacheSharedTtlCleanupManager$CleanupWorker.body(GridCacheSharedTtlCleanupManager.java:178)
>       at 
> org.apache.ignite.internal.util.worker.GridWorker.run(GridWorker.java:120)
>       at java.lang.Thread.run(Thread.java:748)
> {noformat}
> The root cause of the issue is the fact that 
> {{IgniteCacheOffheapManagerImpl#expireInternal}} does not try to acquire a 
> checkpoint read lock at all.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to