You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ignite.apache.org by "Pavel Kuznetsov (JIRA)" <ji...@apache.org> on 2019/02/21 18:28:00 UTC
[jira] [Comment Edited] (IGNITE-6609) H2PkHashIndex.H2Cursor
doesn't take in count expiration
[ https://issues.apache.org/jira/browse/IGNITE-6609?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16774384#comment-16774384 ]
Pavel Kuznetsov edited comment on IGNITE-6609 at 2/21/19 6:27 PM:
------------------------------------------------------------------
p2: do we need to check row for expire time in the {{org.apache.ignite.internal.processors.query.h2.database.H2TreeIndex#find}} ?
{code:java}
if (!cctx.mvccEnabled() && indexType.isPrimaryKey() && lower != null && upper != null &&
tree.compareRows((H2Row)lower, (H2Row)upper) == 0) {
H2Row row = tree.findOne((H2Row)lower, filter(qryCtxRegistry.getThreadLocal()), null);
return (row == null) ? GridH2Cursor.EMPTY : new SingleRowCursor(row);
}
{code}
was (Author: pkouznet):
p2: do we need to check row for expire time in the {{org.apache.ignite.internal.processors.query.h2.database.H2TreeIndex#find}} ?
{code:java}
if (!cctx.mvccEnabled() && indexType.isPrimaryKey() && lower != null && upper != null &&
tree.compareRows((H2Row)lower, (H2Row)upper) == 0) {
H2Row row = tree.findOne((H2Row)lower, filter(qryCtxRegistry.getThreadLocal()), null);
return (row == null) ? GridH2Cursor.EMPTY : new SingleRowCursor(row);
}
{code}
> H2PkHashIndex.H2Cursor doesn't take in count expiration
> -------------------------------------------------------
>
> Key: IGNITE-6609
> URL: https://issues.apache.org/jira/browse/IGNITE-6609
> Project: Ignite
> Issue Type: Bug
> Components: sql
> Affects Versions: 2.3
> Reporter: Vladimir Ozerov
> Assignee: Pavel Kuznetsov
> Priority: Major
> Time Spent: 10m
> Remaining Estimate: 0h
>
> Correct: {{H2Cursor#next}} - expire time is checked;
> Incorrect: {{H2PkHashIndex.H2Cursor#next}} - expired time is ignored.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)