<div dir="ltr">Hi Gunnar,<div><br></div><div>1. Constraint composition: I agree with Guillaume, I thinks is better let the providers do their own implementation.</div><div>2. Email regex: Good for me.</div><div>3. @Positive/@Negative: Ok.</div><div><br></div><div>Thanks!</div><div><br></div><div>Marco</div><div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Mar 24, 2017 at 3:54 PM, Guillaume Smet <span dir="ltr">&lt;<a href="mailto:guillaume.smet@gmail.com" target="_blank">guillaume.smet@gmail.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi Gunnar,<br><div><div class="gmail_extra"><br><div class="gmail_quote"><span class="">On Fri, Mar 24, 2017 at 2:43 PM, Gunnar Morling <span dir="ltr">&lt;<a href="mailto:gunnar@hibernate.org" target="_blank">gunnar@hibernate.org</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
* Should @NotEmpty mandate the usage of constraint composition (as<br>
it&#39;s done in the PRs)? It essentially prescribes an &quot;implementation&quot;,<br>
i.e. providers wouldn&#39;t even need a constraint validator for it. But<br>
then this excludes any more efficient implementation a provider may<br>
have (well, they could by special-handling this constraint).<br></blockquote><div><br></div></span><div>I&#39;m not convinced it&#39;s such a good idea to use constraint composition for @NotBlank and @NotEmpty. I would let the implementation to the providers, especially since using constraint composition does not really buy us anything in this case.<span class="HOEnZb"><font color="#888888"><br><br>-- <br></font></span></div><span class="HOEnZb"><font color="#888888"><div>Guillaume<br></div></font></span></div></div></div></div>
<br>______________________________<wbr>_________________<br>
beanvalidation-dev mailing list<br>
<a href="mailto:beanvalidation-dev@lists.jboss.org">beanvalidation-dev@lists.<wbr>jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/beanvalidation-dev" rel="noreferrer" target="_blank">https://lists.jboss.org/<wbr>mailman/listinfo/<wbr>beanvalidation-dev</a><br></blockquote></div><br></div>