CDV ❯ non-TC version of DistributedCache (ie. LocalCache) throws UnsupportedOperationException

• Type: Bug
• Status: Closed
• Priority: 2 Major
• Resolution: Won't Fix
• Component/s:
• Labels:

• Assignee: teck
• Reporter: teck
• Created: March 19, 2010
• Votes: 0
• Watchers: 0
• Updated:

  March 24, 2011
• Resolved:

  November 02, 2010

Description

tim-distributed-cache/src/main/java/org/terracotta/cache/impl/LocalCache.java

A handful of methods in there throw UnsupportedOperationException (likely needlessly). Review the exceptions and update test that excercises map w/o TC

Comments

Chris Dennis 2010-03-19

One thing to note here is that both values() and containsValue() in the the TC clustered distributed cache implementation throw UnsupportedOperationException as well. My vague recollection is that this was due to concerns about the performance (and possibly the incoherent nature) of these operations on a clustered cache. Hence the UnsupportedOperationException should stay on these methods unless we add implementations to the clustered one too. This doesn’t excuse us from implementing the other methods correctly though.

Fiona OShea 2010-11-02

Does this affect express?

Tim Eck 2010-11-02

This doesn’t really affect express since you can’t (currently) get unclustered instances of the toolkit datatypes.

Given Chris’ comments I’m content to just close this out now since we probably want parity between the clustered and unclustered implementations

Tim Eck 2010-11-02

closing this for now since we want to maintain parity between the clustered and unclustered versions