<div dir="ltr">Yeah, that&#39;s a good point of course. I liked using a map as it very naturally expressed that there may be only one rule per &quot;from&quot; type, but making the API evolvable is more important I guess.<div>
<br></div><div style>How about using ConvertGroup directly, i.e. return a Set&lt;ConvertGroup&gt;? In that context the name doesn&#39;t read that well, but OTOH <span style="font-family:arial,sans-serif;font-size:13px">GroupConvertionDescriptor would more or less resemble ConvertGroup. WDYT?</span></div>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/1/31 Emmanuel Bernard <span dir="ltr">&lt;<a href="mailto:emmanuel@hibernate.org" target="_blank">emmanuel@hibernate.org</a>&gt;</span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I&#39;d rather have a GroupConversionDescriptor representing the from / to.<br>
My reason is that if we add a new attribute to @ConvertGroup later, we<br>
will be able to add it on the descriptor as well.<br>
<br>
I think we tend to use a Set for the collection of elements in the spec.<br>
<br>
So that would become a Set&lt;GroupConvertionDescriptor&gt;.<br>
<span class="HOEnZb"><font color="#888888"><br>
Emmanuel<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
On Wed 2013-01-30 22:39, Gunnar Morling wrote:<br>
&gt; Hi all,<br>
&gt;<br>
&gt; I think one natural way for representing group conversions in the metadata<br>
&gt; API would be to add a method<br>
&gt;<br>
&gt;     Map&lt;Class&lt;?&gt;, Class&lt;?&gt;&gt; getGroupConversions()<br>
&gt;<br>
&gt; to ParameterDescriptor, ReturnValueDescriptor and PropertyDescriptor<br>
&gt; (that&#39;s also how conversions are represented internally in the RI).<br>
&gt;<br>
&gt; Any thoughts on that, or other suggestions? If no one objects, I&#39;d move<br>
&gt; forward and add the method to the spec and API.<br>
&gt;<br>
&gt; --Gunnar<br>
&gt;<br>
&gt;<br>
&gt;<br>
&gt; 2013/1/18 Gunnar Morling &lt;<a href="mailto:gunnar@hibernate.org">gunnar@hibernate.org</a>&gt;<br>
&gt;<br>
&gt; &gt; &gt; We should.<br>
&gt; &gt;<br>
&gt; &gt; Ok, I&#39;ve created <a href="https://hibernate.onjira.com/browse/BVAL-361" target="_blank">https://hibernate.onjira.com/browse/BVAL-361</a> for this.<br>
&gt; &gt;<br>
&gt; &gt; --Gunnar<br>
&gt; &gt;<br>
&gt; &gt;<br>
&gt; &gt; 2013/1/18 Emmanuel Bernard &lt;<a href="mailto:emmanuel@hibernate.org">emmanuel@hibernate.org</a>&gt;<br>
&gt; &gt;<br>
&gt; &gt;&gt; We should.<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; The reason the metadata API is complete is so that non Java system can<br>
&gt; &gt;&gt; propagate the constraints and logic of validation. Client side presentation<br>
&gt; &gt;&gt; framework was the canonical example.<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; On 17 janv. 2013, at 21:44, Gunnar Morling &lt;<a href="mailto:gunnar@hibernate.org">gunnar@hibernate.org</a>&gt; wrote:<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; &gt; Hi all,<br>
&gt; &gt;&gt; &gt;<br>
&gt; &gt;&gt; &gt; As the title says: should we provide a way to access configured group<br>
&gt; &gt;&gt; conversions via the meta-data API?<br>
&gt; &gt;&gt; &gt;<br>
&gt; &gt;&gt; &gt; I&#39;m undecided, I don&#39;t really see where this would be useful, OTOH<br>
&gt; &gt;&gt; we&#39;ve exposed all BV-related configurations via the meta-data API so far,<br>
&gt; &gt;&gt; so I guess we probably should for the sake of completeness.<br>
&gt; &gt;&gt; &gt;<br>
&gt; &gt;&gt; &gt; Thoughts?<br>
&gt; &gt;&gt; &gt;<br>
&gt; &gt;&gt; &gt; --Gunnar<br>
&gt; &gt;&gt; &gt;<br>
&gt; &gt;&gt; &gt; _______________________________________________<br>
&gt; &gt;&gt; &gt; beanvalidation-dev mailing list<br>
&gt; &gt;&gt; &gt; <a href="mailto:beanvalidation-dev@lists.jboss.org">beanvalidation-dev@lists.jboss.org</a><br>
&gt; &gt;&gt; &gt; <a href="https://lists.jboss.org/mailman/listinfo/beanvalidation-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/beanvalidation-dev</a><br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; _______________________________________________<br>
&gt; &gt;&gt; beanvalidation-dev mailing list<br>
&gt; &gt;&gt; <a href="mailto:beanvalidation-dev@lists.jboss.org">beanvalidation-dev@lists.jboss.org</a><br>
&gt; &gt;&gt; <a href="https://lists.jboss.org/mailman/listinfo/beanvalidation-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/beanvalidation-dev</a><br>
&gt; &gt;&gt;<br>
&gt; &gt;<br>
&gt; &gt;<br>
<br>
&gt; _______________________________________________<br>
&gt; beanvalidation-dev mailing list<br>
&gt; <a href="mailto:beanvalidation-dev@lists.jboss.org">beanvalidation-dev@lists.jboss.org</a><br>
&gt; <a href="https://lists.jboss.org/mailman/listinfo/beanvalidation-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/beanvalidation-dev</a><br>
<br>
_______________________________________________<br>
beanvalidation-dev mailing list<br>
<a href="mailto:beanvalidation-dev@lists.jboss.org">beanvalidation-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/beanvalidation-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/beanvalidation-dev</a><br>
</div></div></blockquote></div><br></div>