<div dir="ltr">Would that take affect for JDG as well in a future release?<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Sep 8, 2016 at 2:06 PM, Tristan Tarrant <span dir="ltr"><<a href="mailto:ttarrant@redhat.com" target="_blank">ttarrant@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Ok, I've checked:<br>
<br>
the as-modules/client module doesn't really have any reason to exist,<br>
since the as-modules/embedded one contains everything already (and is<br>
misnamed).<br>
Also the problem about the single module is only on the remote side,<br>
which doesn't re-export any of the APIs. The embedded<br>
"org.infinispan.main:x.y" module does it correctly, but I don't like the<br>
name (as it create confusion with the usual "main" slot").<br>
<br>
My proposal:<br>
- remove as-modules/client<br>
- move as-modules/embedded to become as-modules<br>
- for symmetry with the uberjars, have an org.infinispan.embedded and<br>
org.infinispan.remote modules which re-export the appropriate APIs.<br>
<br>
<a href="https://issues.jboss.org/browse/ISPN-7007" rel="noreferrer" target="_blank">https://issues.jboss.org/<wbr>browse/ISPN-7007</a><br>
<span class="HOEnZb"><font color="#888888"><br>
Tristan<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
<br>
On 08/09/16 13:22, Sanne Grinovero wrote:<br>
> +1<br>
><br>
> But please also add a couple of "user facing" modules which expose all<br>
> the necessary components as a single unit.<br>
><br>
> For my use case in Hibernate OGM I needed to list all of:<br>
><br>
> <module name="org.infinispan.client.<wbr>hotrod"<br>
> slot="${infinispan.module.<wbr>slot}" /><br>
> <module name="org.infinispan.<wbr>protostream"<br>
> slot="${infinispan.module.<wbr>slot}" /><br>
> <module name="org.infinispan.commons"<br>
> slot="${infinispan.module.<wbr>slot}" /><br>
> <module name="org.infinispan.query.<wbr>dsl"<br>
> slot="${infinispan.module.<wbr>slot}" /><br>
> <module name="org.infinispan.query.<wbr>remote.client"<br>
> slot="${infinispan.module.<wbr>slot}" /><br>
><br>
> (figured this list out by trial & error)<br>
><br>
> Would be nice to have a single module ID for "remote client" and a<br>
> single module ID for "embedded usage", document these clearly, and<br>
> then mark the other modules as private API.<br>
><br>
> <properties><br>
> <property name="jboss.api" value="private"/><br>
> </properties><br>
><br>
> Thanks,<br>
> Sanne<br>
><br>
><br>
> On 8 September 2016 at 08:17, Sebastian Laskawiec <<a href="mailto:slaskawi@redhat.com">slaskawi@redhat.com</a>> wrote:<br>
>> Technically I'm all for it.<br>
>><br>
>> The question that I have is how many users/clients do we have who wants to<br>
>> use only RemoteCacheManager?<br>
>><br>
>> Thanks<br>
>> Sebastian<br>
>><br>
>> On Thu, Sep 8, 2016 at 9:08 AM, Gustavo Fernandes <<a href="mailto:gustavo@infinispan.org">gustavo@infinispan.org</a>><br>
>> wrote:<br>
>>> Actually, looking at the current modules zip listed in the website [1], I<br>
>>> can see that protostream and hotrod-client modules are are there,<br>
>>> together with embedded modules (core, lucene-directoty, etc), isn't that<br>
>>> correct? What is in the other unlisted "client" module?<br>
>>><br>
>>><br>
>>> [1]<br>
>>> <a href="http://downloads.jboss.org/infinispan/9.0.0.Alpha4/infinispan-as-embedded-modules-9.0.0.Alpha4.zip" rel="noreferrer" target="_blank">http://downloads.jboss.org/<wbr>infinispan/9.0.0.Alpha4/<wbr>infinispan-as-embedded-<wbr>modules-9.0.0.Alpha4.zip</a><br>
>>><br>
>>> On Thu, Sep 8, 2016 at 7:59 AM, Gustavo Fernandes <<a href="mailto:gustavo@infinispan.org">gustavo@infinispan.org</a>><br>
>>> wrote:<br>
>>>> +1<br>
>>>><br>
>>>><br>
>>>> On Thu, Sep 8, 2016 at 7:37 AM, Tristan Tarrant <<a href="mailto:ttarrant@redhat.com">ttarrant@redhat.com</a>><br>
>>>> wrote:<br>
>>>>> Hi all,<br>
>>>>><br>
>>>>> we currently distribute two separate packages for WildFly modules:<br>
>>>>> embedded and client. Unfortunately we only list the former on the<br>
>>>>> download page.<br>
>>>>> My proposal is to combine the two packages into one. WDYT ?<br>
>>>>><br>
>>>>> Tristan<br>
>>>>> --<br>
>>>>> Tristan Tarrant<br>
>>>>> Infinispan Lead<br>
>>>>> JBoss, a division of Red Hat<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>
>>>><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>
>><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>
> ______________________________<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>
><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>
</div></div></blockquote></div><br></div>