<div dir="ltr">+1<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jun 9, 2017 at 11:02 PM, Radim Vansa <span dir="ltr"><<a href="mailto:rvansa@redhat.com" target="_blank">rvansa@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi guys,<br>
<br>
when the functional API has been outline, the interfaces were put into<br>
infinispan-commons to make it possible to share these between remote<br>
clients and embedded use case. However, it seems that reusing this as-is<br>
impossible or at least impractical as we cannot send the lambdas in a<br>
language neutral way. In the future, we may implement a way to share<br>
functions between client and a server but that will most likely result<br>
in an interface accepting something else than Function<ReadWriteEntry,<br>
R>. Also, it's rather weird to have two EntryVersion interfaces.<br>
<br>
Therefore I suggest moving org.infinispan.commons.api.<wbr>functional to<br>
infinispan-core, package org.infinispan.api.functional<br>
<br>
You might say that the server-side code would use the interfaces, but<br>
once it's running on server, it should depend on core (or core-api) -<br>
commons is what is shared with the client, and if the client will in<br>
future register a new function on the server, the user code should<br>
depend on core-api as well (client-hotrod itself does not have to).<br>
<br>
If you wonder what led me to this is that I've tried to add<br>
SerializableFunction overloads to the FunctionalMap and found out that<br>
SerializableFunction et all are only in infinispan-core (for good).<br>
<br>
Please let me know if you have objections/if there something I have missed.<br>
<span class="HOEnZb"><font color="#888888"><br>
Radim<br>
<br>
--<br>
Radim Vansa <<a href="mailto:rvansa@redhat.com">rvansa@redhat.com</a>><br>
JBoss Performance Team<br>
<br>
______________________________<wbr>_________________<br>
infinispan-dev mailing list<br>
<a href="mailto:infinispan-dev@lists.jboss.org">infinispan-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/infinispan-dev" rel="noreferrer" target="_blank">https://lists.jboss.org/<wbr>mailman/listinfo/infinispan-<wbr>dev</a><br>
</font></span></blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><p style="font-weight:bold;margin:0;padding:0;font-size:14px;text-transform:uppercase;margin-bottom:0"><span>Vittorio</span> <span>Rigamonti</span></p>
<p style="font-weight:normal;font-size:10px;margin:0px 0px 4px;text-transform:uppercase"><span>Senior Software Engineer</span></p>
<p style="font-weight:normal;margin:0;font-size:10px;color:#999"><a style="color:#0088ce;font-size:10px;margin:0;text-decoration:none;font-family:'overpass',sans-serif" href="https://www.redhat.com" target="_blank">Red Hat <span><br><br></span></a></p>
<span style="font-size:10px;margin:0px;color:rgb(153,153,153)"><p style="font-size:10px;margin:0">Milan, Italy</p></span>
<p style="font-weight:normal;margin:0px 0px 6px;font-size:10px;color:rgb(153,153,153)"><span style="margin:0px;padding:0px">
<a style="color:#0088ce;font-size:10px;margin:0;text-decoration:none;font-family:'overpass',sans-serif" href="mailto:vrigamon@redhat.com" target="_blank">vrigamon@redhat.com</a><br></span></p><p style="font-weight:normal;margin:0px 0px 6px;font-size:10px;color:rgb(153,153,153)"><span style="margin:0px;padding:0px">irc: rigazilla </span>
</p>
<a href="https://red.ht/sig" target="_blank"> <img src="https://www.redhat.com/profiles/rh/themes/redhatdotcom/img/logo-red-hat-black.png" width="90" height="auto"></a></div></div></div></div>
</div>