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

Anton Vinogradov commented on IGNITE-2710:
------------------------------------------

Roman,

1) I think that  "if (cache.get(ses.getId()) == null) {" should be replaced by 
isValid() check because cache.get if much more heavy than reading volatile.
2) Also new session should be created on demand at any cases, not only if 
current invalidated? Possible previous one should be invalidated in case new 
one demanded.

> Session not unbind from current request after invoking 
> request.getSession().invalidate()
> ----------------------------------------------------------------------------------------
>
>                 Key: IGNITE-2710
>                 URL: https://issues.apache.org/jira/browse/IGNITE-2710
>             Project: Ignite
>          Issue Type: Bug
>          Components: cache
>    Affects Versions: 1.5.0.final
>         Environment: java8, tomcat8
>            Reporter: YuxuanWang
>            Assignee: Roman Shtykh
>              Labels: community, session, user
>             Fix For: 1.6
>
>
> System.out.println(request.getSession().getId());
> request.getSession().invalidate();
> System.out.println(request.getSession().getId());
> getSession() although return the same session after the invalidation.



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

Reply via email to