[ 
https://issues.apache.org/jira/browse/IGNITE-5781?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Aleksandr Meterko reassigned IGNITE-5781:
-----------------------------------------

    Assignee: Aleksandr Meterko

> Visor throws ClassCastException if cache store implementation is other than 
> CacheJdbcPojoStore
> ----------------------------------------------------------------------------------------------
>
>                 Key: IGNITE-5781
>                 URL: https://issues.apache.org/jira/browse/IGNITE-5781
>             Project: Ignite
>          Issue Type: Bug
>          Components: visor
>    Affects Versions: 2.0
>            Reporter: Valentin Kulichenko
>            Assignee: Aleksandr Meterko
>             Fix For: 2.2
>
>
> Issue is reported on user list: 
> http://apache-ignite-users.70518.x6.nabble.com/Problem-with-Visor-and-Cassandra-Cache-Store-td15076.html
> There is an obvious bug in the code. {{VisorCacheJdbcType#list}} method 
> checks the type of store factory like this:
> {code}
> if (factory != null || factory instanceof CacheJdbcPojoStoreFactory) {
>     CacheJdbcPojoStoreFactory jdbcFactory = (CacheJdbcPojoStoreFactory) 
> factory;
> {code}
> It should be {{&&}} instead of {{||}}, because otherwise condition will be 
> {{true}} for any factory that is not {{null}}. Even better if {{factory != 
> null}} is removed completely as {{instanceof}} returns {{false}} for {{null}} 
> values anyway.
> However, it's not clear to me why this scenario is reproduced only in certain 
> conditions (see mailing list thread for details). It's possible that there is 
> another hidden bug, this needs to be investigated.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to