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

Alexey Goncharuk commented on IGNITE-8237:
------------------------------------------

[~kukushal],

Please remove the property check logic outside of ServerImpl, it is not a right 
place to run this check. Note that GridCacheProcessor already receives 
deserialized joining node data. It should throw an exception if validation 
failed. If needed, add a separate callback for all components which will be 
able to throw an exception.

Also, system property boolean check using string comparison looks odd, I think 
we should use Boolean.getBoolean(String) which is a usual property check.

> Ignite blocks on SecurityException in exchange-worker due to unauthorised 
> on-heap cache configuration 
> ------------------------------------------------------------------------------------------------------
>
>                 Key: IGNITE-8237
>                 URL: https://issues.apache.org/jira/browse/IGNITE-8237
>             Project: Ignite
>          Issue Type: Bug
>    Affects Versions: 2.5
>            Reporter: Alexey Kukushkin
>            Assignee: Alexey Kukushkin
>            Priority: Blocker
>              Labels: MakeTeamcityGreenAgain
>             Fix For: 2.5
>
>
> Ignite blocks on SecurityException in exchange-worker due to unauthorised 
> on-heap cache configuration. Consider moving IGNITE_DISABLE_ONHEAP_CACHE 
> system property check to a more appropriate place to avoid blocking Ignite.



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

Reply via email to