[ https://issues.apache.org/jira/browse/IGNITE-5781?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16095817#comment-16095817 ]
ASF GitHub Bot commented on IGNITE-5781: ---------------------------------------- Github user Desperus closed the pull request at: https://github.com/apache/ignite/pull/2321 > 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)