[infinispan-issues] [JBoss JIRA] (ISPN-8204) Remove should be conditional

[ https://issues.jboss.org/browse/ISPN-8204?page=com.atlassian.jira.plugin.... ] Pedro Ruivo commented on ISPN-8204: ----------------------------------- There are many changes in semantic. {{IRV}} flag with non-condition removal will create a {{RemoveCommand}} with {{LoadType}} {{DONT_LOAD}}: * the context entry isn't mark as read (skips WriteSkew check) * the value isn't loaded from cache loader if it doesn't exists in {{DataContainer}} * if the entry doesn't exists in context, it won't be loaded from remote node * it doesn't create a {{SuccessfulResponse}} (doesn't send the return value) * in a transaction, it doesn't perform the remote get. My point is that we lose all the {{IRV}} behavior, and that isn't good IMO. If we go we this changes, we can remove the flag (it is only used in {{put(k,v)}} and create a {{set(k,v)}} method. At least, it can remove a couple of if-branches in the code... Also, I would like to know what others think about it. If I'm the only one thinking this way, there is no point in waste time to discuss it :) -- This message was sent by Atlassian JIRA (v7.2.3#72005)