ignite-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alexey Kuznetsov (JIRA)" <j...@apache.org>
Subject [jira] [Comment Edited] (IGNITE-1161) Close rest sql cursor after delay
Date Wed, 04 Nov 2015 10:00:36 GMT

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

Alexey Kuznetsov edited comment on IGNITE-1161 at 11/4/15 10:00 AM:
--------------------------------------------------------------------

Andrey, please rename variable in one way. Currently I see that we use names like: "val, t,
cur" for accessing same type.

Also please rework code like "lock().lock()" may be we could simply extend ReentrantLock directly?

Double check if "fetch next page" update query last time access.
Also check TC.


was (Author: kuaw26):
Andrey, please rename variable in one way. Currently I see that we use names like: "val, t,
cur" for accessing same type.

Also please rework code like "lock().lock()" may be we could simply extend ReentrantLock directly?

Double check if "fetch next page" update query last time access.

> Close rest sql cursor after delay 
> ----------------------------------
>
>                 Key: IGNITE-1161
>                 URL: https://issues.apache.org/jira/browse/IGNITE-1161
>             Project: Ignite
>          Issue Type: Task
>          Components: general
>            Reporter: Irina Vasilinets
>            Assignee: Andrey Novikov
>             Fix For: 1.5
>
>         Attachments: ignite-1161.patch, master_ignite-1161.patch, master_ignite-1161.patch
>
>
> QueryCommandHandler stores cursors for all queries that was executed by rest. Old queries
cursors should be closed after delay and should be removed from the map.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message