[infinispan-issues] [JBoss JIRA] (ISPN-9027) Distinguishing multiple server Store configurations is impossible

Distinguishing multiple server Store configurations is impossible ----------------------------------------------------------------- Key: ISPN-9027 URL: https://issues.jboss.org/browse/ISPN-9027 Project: Infinispan Issue Type: Bug Components: Loaders and Stores, Server Affects Versions: 9.2.1.Final Reporter: Sebastian Laskawiec Assignee: Ryan Emerson Priority: Major Fix For: 10.0.0.Beta4, 9.4.14.Final Attachments: clustered.xml h3. Problem description One of our users reported that when using two Deployed Cache Stores, their configuration gets overridden and both behaves as if they had the same configuration. Here's an example: {code} <distributed-cache name="cassandracache1" owners="2" segments="256" mode="SYNC"> <store name="store1" class="org.infinispan.persistence.cassandra.CassandraStore" preload="true"> <property name="autoCreateKeyspace"> true </property> <property name="keyspace"> Infinispan </property> <property name="entryTable"> InfinispanEntries </property> <property name="servers"> 172.17.0.2[9042] </property> </store> </distributed-cache> <distributed-cache name="cassandracache2" owners="2" segments="256" mode="SYNC"> <store name="store2" class="org.infinispan.persistence.cassandra.CassandraStore" preload="true"> <property name="autoCreateKeyspace"> true </property> <property name="keyspace"> Infinispan </property> <property name="entryTable"> InfinispanEntries1 </property> <property name="servers"> 172.17.0.2[9042] </property> </store> </distributed-cache> {code} Both caches ({{cassandracache1}} and {{cassandracache2}}) use the same {{entryTable}} which is set to {{InfinispanEntries1}}. h3. Investigation {{CacheStoreFactory}} implementation were created to fabricate Loader/Writer instance based on parsed configuration (via {{createInstance}} method). This method receives from {{PersistenceManagerImpl}} an instance of an {{AbstractStoreConfiguration}} (here's an example (1)) two times - once per each parsed configuration. The parsing part seems OK but we do not parse Cache Store Name, which makes differentiating both configuration impossible. h3. Proposed fix * Add {{name}} attribute to {{StoreConfiguration}}. * Either add an explicit parameter to {{CacheStoreFactory#createInstance(StoreConfiguration cfg, String cacheStoreName)}} or scan for Cache Store name in both implementations ({{DeployedCacheStoreFactory}} and {{LocalClassLoaderCacheStoreFactory}}. {code} (1) AbstractStoreConfiguration [attributes=DeployedStoreConfiguration = [fetchPersistentState=false, purgeOnStartup=false, ignoreModifications=false, preload=true, shared=false, transactional=false, maxBatchSize=100, properties={keyspace=Infinispan, connectionPool.poolTimeoutMillis=5, entryTable=InfinispanEntries, connectionPool.idleTimeoutSeconds=120, connectionPool.heartbeatIntervalSeconds=30, autoCreateKeyspace=true, servers=172.17.0.2[9042]}, customStoreClassName=org.infinispan.persistence.cassandra.CassandraStore], async=AsyncStoreConfiguration [attributes=AsyncStoreConfiguration = [enabled=false, modificationQueueSize=1024, threadPoolSize=1]], singletonStore=SingletonStoreConfiguration [attributes=SingletonStoreConfiguration = [enabled=false, push-state-timeout=10000, push-state-when-coordinator=true]]] {code}