cache.put("key",null) fails [DATACOUCH-628]

[spring-projects-issues]

achoice opened DATACOUCH-628 and commented

Caching of null values fail with exception thrown by Couchbase client.

CouchbaseCacheConfiguration.defaultCacheConfig() seems that null valus should be cachable. https://github.com/spring-projects/spring-data-couchbase/blob/master/src/main/java/org/springframework/data/couchbase/cache/CouchbaseCacheConfiguration.java#L54

 

@Test
public void cacheNull() {
    final String CACHE_NAME = "CACHE_TEST";
    
    final CouchbaseCacheConfiguration defaultCacheConfig = CouchbaseCacheConfiguration.defaultCacheConfig();
    assertTrue(defaultCacheConfig.getAllowCacheNullValues()); // expect null values to be cachable now

    final CouchbaseCacheManager cacheManager = CouchbaseCacheManager
            .builder(couchbaseClientFactory)
            .cacheDefaults(defaultCacheConfig)
            .initialCacheNames(Set.of(CACHE_NAME))
            .build();
    cacheManager.afterPropertiesSet();

    // TEST cache null
    final Cache cache = cacheManager.getCache(CACHE_NAME);
    cache.put("KEY", null); // exception thrown by couchbase client
}

 

---

Affects: 4.0.4 (Neumann SR4)

[spring-projects-issues]

Michael Reiche commented

I've filed an issue against the underlying api - https://issues.couchbase.com/browse/JCBC-1723

[spring-projects-issues]

achoice commented

Thanks for quick attention. Perhaps not so easy to solve if underlying api does not allow upset of nulls. (Perhaps older client supported null values? Strange how this has gone unnoticed)

In Spring docs [1] I found this: "Note: Due to the generic use of caching, it is recommended that implementations allow storage of null values (for example to cache methods that return null)." so if this cant support caching of nulls it should be clearly stated/documented.

[1] https://docs.spring.io/spring-framework/docs/current/javadoc-api/org/springframework/cache/Cache.html