<div dir="ltr">Hi all,<div><br></div><div>As you might know I&#39;m working since my arrival, among other things, on ISPN-5728 Jira [1], where the idea is to override the default ConcurrentMap methods that are missing in CacheImpl (merge, replaceAll, compute ... )</div><div><br></div><div>I&#39;ve created a pull-request [2] for compute, computeIfAbsent and computeIfPresent methods, creating two new commands. By the way, I did the same thing for the merge method in a branch that I haven&#39;t pull requested yet.</div><div><br></div><div>There is an opposite view between Radim and Will concerning the implementation of these methods. To make it short :</div><div>In one side Will considers compute/merge best implementation should be as a new Command (so what is already done)</div><div>In the other side, Radim considers adding another command is not necessary as we could simple implement these methods using <span style="background-color:rgba(27,31,35,0.0470588);color:rgb(36,41,46);font-family:sfmono-regular,consolas,&quot;liberation mono&quot;,menlo,courier,monospace;font-size:11.9px">ReadWriteKeyCommand</span></div><div><span style="background-color:rgba(27,31,35,0.0470588);color:rgb(36,41,46);font-family:sfmono-regular,consolas,&quot;liberation mono&quot;,menlo,courier,monospace;font-size:11.9px"><br></span></div><div>The detailed discussion and arguments of both sides is on GitHub [2]</div><div><br></div><div>Before moving forward and making any choice by myself, I would like to hear your opinions. For the record, it doesn&#39;t bother me redoing everything if most people think like Radim because working on commands has helped me to learn and understand more about infinispan internals, so this hasn&#39;t been a waste of time for me.</div><div><br></div><div>Katia</div><div><br></div><div>[1] <a href="https://issues.jboss.org/browse/ISPN-5728">https://issues.jboss.org/browse/ISPN-5728</a></div><div>[2] <a href="https://github.com/infinispan/infinispan/pull/5046">https://github.com/infinispan/infinispan/pull/5046</a></div><div><br></div></div>