[
https://issues.apache.org/jira/browse/IGNITE-8747?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16506888#comment-16506888
]
ASF GitHub Bot commented on IGNITE-8747:
----------------------------------------
GitHub user AMashenkov opened a pull request:
https://github.com/apache/ignite/pull/4165
IGNITE-8747: Remove\RemoveAll shouldn't report of successful deletion for
expired entries.
(cherry picked from commit f65f0a5)
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/gridgain/apache-ignite ignite-8747
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/ignite/pull/4165.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #4165
----
commit ec567c0e5bd0b13f2ea9e097057c2cb4b57f2467
Author: Andrey V. Mashenkov <andrey.mashenkov@...>
Date: 2018-06-09T08:52:38Z
IGNITE-8747: Apply AccessPolicy instantly on first touch.
(cherry picked from commit f65f0a5)
----
> AccessExpiryPolicy should be check on first entry touch.
> --------------------------------------------------------
>
> Key: IGNITE-8747
> URL: https://issues.apache.org/jira/browse/IGNITE-8747
> Project: Ignite
> Issue Type: Improvement
> Components: cache
> Reporter: Andrew Mashenkov
> Assignee: Andrew Mashenkov
> Priority: Major
> Labels: MakeTeamcityGreenAgain, tck, test-failure
>
> We have 2 TCK 1.0 test that are passed due to we have eagerTtl=true by
> default.
> The reason is remove() return true even if an expired entry was removed.
> Seems, we have to evict expired entry from cache on remove(), but do not
> count it as removed.
> java.lang.AssertionError
> at
> org.jsr107.tck.expiry.CacheExpiryTest.expire_whenAccessed(CacheExpiryTest.java:326)
> java.lang.AssertionError: expected:<0> but was:<1> at
> org.jsr107.tck.expiry.CacheExpiryTest.testCacheStatisticsRemoveAll(CacheExpiryTest.java:160)
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
