[jboss-jira] [JBoss JIRA] Updated: (JBCACHE-948) Patch to add return value (boolean) to removeChild*() and removeNode*() methods

Patch to add return value (boolean) to removeChild*() and removeNode*() methods ------------------------------------------------------------------------------- Key: JBCACHE-948 URL: http://jira.jboss.com/jira/browse/JBCACHE-948 Project: JBoss Cache Issue Type: Patch Security Level: Public(Everyone can see) Affects Versions: 2.0.0.ALPHA1, 2.0.0.ALPHA2 Reporter: Elias Ross Assigned To: Manik Surtani Attachments: remove.patch It is useful to know if a node was actually removed or not. Having a return value adds a bit of usefulness. A boolean return value is consistent with Collections.remove(). In any case, it's a small but potentially useful API change which I suggest for 2.0 release. For example, in CacheImpl, there was no way to detect removal failures: void _remove() ... try { _remove(tx, tmp, create_undo_ops, true, eviction); } catch (Exception e) { log.error("failure removing node " + tmp); } The patch includes the changes and additional tests.