[jira] [Commented] (IGNITE-7972) NPE in TTL manager.

2018-04-16 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on IGNITE-7972:


Github user asfgit closed the pull request at:

https://github.com/apache/ignite/pull/3810


> NPE in TTL manager.
> ---
>
> Key: IGNITE-7972
> URL: https://issues.apache.org/jira/browse/IGNITE-7972
> Project: Ignite
>  Issue Type: Bug
>  Components: cache
>Affects Versions: 2.3
>Reporter: Andrew Mashenkov
>Assignee: Andrew Mashenkov
>Priority: Major
> Fix For: 2.5
>
> Attachments: npe.log
>
>
> TTL manager can try to evict expired entries on cache that wasn't initialized 
> yet due to a race.
> This lead to NPE in unwindEvicts method.
> PFA stacktrace.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (IGNITE-7972) NPE in TTL manager.

2018-04-16 Thread Ivan Rakov (JIRA)

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

Ivan Rakov commented on IGNITE-7972:


I'm ok with this fix, please merge.

> NPE in TTL manager.
> ---
>
> Key: IGNITE-7972
> URL: https://issues.apache.org/jira/browse/IGNITE-7972
> Project: Ignite
>  Issue Type: Bug
>  Components: cache
>Affects Versions: 2.3
>Reporter: Andrew Mashenkov
>Assignee: Andrew Mashenkov
>Priority: Major
> Fix For: 2.5
>
> Attachments: npe.log
>
>
> TTL manager can try to evict expired entries on cache that wasn't initialized 
> yet due to a race.
> This lead to NPE in unwindEvicts method.
> PFA stacktrace.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (IGNITE-7972) NPE in TTL manager.

2018-04-16 Thread Dmitriy Pavlov (JIRA)

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

Dmitriy Pavlov commented on IGNITE-7972:


[~amashenkov], TC looks as red as usual, seems there are no failures related to 
this change (3810)

[~ivan.glukos], could you please look to this change?

> NPE in TTL manager.
> ---
>
> Key: IGNITE-7972
> URL: https://issues.apache.org/jira/browse/IGNITE-7972
> Project: Ignite
>  Issue Type: Bug
>  Components: cache
>Affects Versions: 2.3
>Reporter: Andrew Mashenkov
>Assignee: Andrew Mashenkov
>Priority: Major
> Fix For: 2.5
>
> Attachments: npe.log
>
>
> TTL manager can try to evict expired entries on cache that wasn't initialized 
> yet due to a race.
> This lead to NPE in unwindEvicts method.
> PFA stacktrace.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (IGNITE-7972) NPE in TTL manager.

2018-04-16 Thread Andrew Mashenkov (JIRA)

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

Andrew Mashenkov commented on IGNITE-7972:
--

[~dpavlov], here are all failed tests [1]

[1] 
https://ci.ignite.apache.org/project.html?projectId=IgniteTests24Java8=problems_IgniteTests24Java8=pull%2F3810%2Fhead

> NPE in TTL manager.
> ---
>
> Key: IGNITE-7972
> URL: https://issues.apache.org/jira/browse/IGNITE-7972
> Project: Ignite
>  Issue Type: Bug
>  Components: cache
>Affects Versions: 2.3
>Reporter: Andrew Mashenkov
>Assignee: Andrew Mashenkov
>Priority: Major
> Fix For: 2.5
>
> Attachments: npe.log
>
>
> TTL manager can try to evict expired entries on cache that wasn't initialized 
> yet due to a race.
> This lead to NPE in unwindEvicts method.
> PFA stacktrace.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (IGNITE-7972) NPE in TTL manager.

2018-04-13 Thread Dmitriy Pavlov (JIRA)

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

Dmitriy Pavlov commented on IGNITE-7972:


Could you provide link for convinience?

> NPE in TTL manager.
> ---
>
> Key: IGNITE-7972
> URL: https://issues.apache.org/jira/browse/IGNITE-7972
> Project: Ignite
>  Issue Type: Bug
>  Components: cache
>Affects Versions: 2.3
>Reporter: Andrew Mashenkov
>Assignee: Andrew Mashenkov
>Priority: Major
> Fix For: 2.5
>
> Attachments: npe.log
>
>
> TTL manager can try to evict expired entries on cache that wasn't initialized 
> yet due to a race.
> This lead to NPE in unwindEvicts method.
> PFA stacktrace.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (IGNITE-7972) NPE in TTL manager.

2018-04-13 Thread Andrew Mashenkov (JIRA)

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

Andrew Mashenkov commented on IGNITE-7972:
--

TC tests looks fine.

> NPE in TTL manager.
> ---
>
> Key: IGNITE-7972
> URL: https://issues.apache.org/jira/browse/IGNITE-7972
> Project: Ignite
>  Issue Type: Bug
>  Components: cache
>Affects Versions: 2.3
>Reporter: Andrew Mashenkov
>Assignee: Andrew Mashenkov
>Priority: Major
> Fix For: 2.5
>
> Attachments: npe.log
>
>
> TTL manager can try to evict expired entries on cache that wasn't initialized 
> yet due to a race.
> This lead to NPE in unwindEvicts method.
> PFA stacktrace.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (IGNITE-7972) NPE in TTL manager.

2018-04-12 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on IGNITE-7972:


GitHub user AMashenkov opened a pull request:

https://github.com/apache/ignite/pull/3810

IGNITE-7972: Fixed NPE in TTL manager on unwindEvicts.



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/gridgain/apache-ignite ignite-7972

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/ignite/pull/3810.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 #3810






> NPE in TTL manager.
> ---
>
> Key: IGNITE-7972
> URL: https://issues.apache.org/jira/browse/IGNITE-7972
> Project: Ignite
>  Issue Type: Bug
>  Components: cache
>Affects Versions: 2.3
>Reporter: Andrew Mashenkov
>Assignee: Andrew Mashenkov
>Priority: Major
> Fix For: 2.5
>
> Attachments: npe.log
>
>
> TTL manager can try to evict expired entries on cache that wasn't initialized 
> yet due to a race.
> This lead to NPE in unwindEvicts method.
> PFA stacktrace.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (IGNITE-7972) NPE in TTL manager.

2018-04-12 Thread Andrew Mashenkov (JIRA)

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

Andrew Mashenkov commented on IGNITE-7972:
--

GridCacheUtils.unwindEvicts() takes cache contexts from SharedContext and tries 
to evict entries from known caches.

Possible issue here is that cache context is published to SharedContext before 
it's managers (incl. TtlManager) has started.
There is a guarantee that if cache context.started()==true then all it's 
managers is started as well.

Seems, GridCacheUtils.unwindEvicts() should just check if cache context is 
started.

> NPE in TTL manager.
> ---
>
> Key: IGNITE-7972
> URL: https://issues.apache.org/jira/browse/IGNITE-7972
> Project: Ignite
>  Issue Type: Bug
>  Components: cache
>Affects Versions: 2.3
>Reporter: Andrew Mashenkov
>Assignee: Andrew Mashenkov
>Priority: Major
> Attachments: npe.log
>
>
> TTL manager can try to evict expired entries on cache that wasn't initialized 
> yet due to a race.
> This lead to NPE in unwindEvicts method.
> PFA stacktrace.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)