Vladsz83 commented on code in PR #13599:
URL: https://github.com/apache/ignite/pull/13599#discussion_r4445434895


##########
modules/core/src/main/java/org/apache/ignite/internal/processors/cache/GridCacheMapEntry.java:
##########
@@ -2526,6 +2569,18 @@ private boolean skipInterceptor(@Nullable 
GridCacheVersion explicitVer) {
 
             long expTime = expireTime < 0 ? CU.toExpireTime(ttl) : expireTime;
 
+            CacheDataRow expiredRow = null;
+
+            // The value is already expired: store it as removed instead of 
storing an already expired row.
+            // A row with expire time in the past must never be written to the 
row store (see IGNITE-25194).

Review Comment:
   Do we need to link done tickets without TODO?



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to