EHC ❯ Cache.getAllWithLoader always returns null for expired elements

• Type: Bug
• Status: Closed
• Priority: 2 Major
• Resolution: Fixed
• Component/s: ehcache-core
• Labels:

• Assignee: lorban
• Reporter: cpruett
• Created: November 18, 2010
• Votes: 0
• Watchers: 0
• Updated:

  March 05, 2013
• Resolved:

  January 26, 2011

Description

When getAllWithLoader() encounters expired elements, it populates the return map with null values instead of delegating to the cache loader to reload the value. This behavior is asymmetric with getWithLoader(), which correctly calls the loader when expired elements are found.

I believe that the following code is the culprit:

public Map getAllWithLoader(Collection keys, Object loaderArgument) throws CacheException {
	// ... snipped ...
	if (isKeyInCache(key)) {
	    Element element = get(key);
	    if (element != null) {
	        map.put(key, element.getObjectValue());
	    } else {
	        map.put(key, null);
	    }
	} else {
	    missingKeys.add(key);
	}
	// ... snipped ...

}

isKeyInCache() returns true, regardless of whether the key is expired.

Comments

Ludovic Orban 2011-01-26

Fixed.