I agree, looks reasonable.  :)  There is also some stuff at
http://svn.apache.org/viewvc/bval/sandbox/jsr303-impl-bench/ if you had any
interest in merging your work there.

Matt


On Mon, Aug 12, 2013 at 6:58 AM, Romain Manni-Bucau
<[email protected]>wrote:

> Hi
>
> just did a simple test:
> https://gist.github.com/rmannibucau/6dab2d80ea2b26c40876
>
> with:
>
> public class ValidatedBeanL1 {
>     @Valid
>     private ValidatedBeanL2 l2;
>
>     @Min(1)
>     private int size;
> }
>
> and
>
>
> public class ValidatedBeanL2 {
>     @NotNull
>     private String name;
>
>     @Size(min = 3, max = 5)
>     private String fullname;
> }
>
> It doesn't look bad to me but wdyt?
>
> *Romain Manni-Bucau*
> *Twitter: @rmannibucau <https://twitter.com/rmannibucau>*
> *Blog: **http://rmannibucau.wordpress.com/*<http://rmannibucau.wordpress.com/>
> *LinkedIn: **http://fr.linkedin.com/in/rmannibucau*
> *Github: https://github.com/rmannibucau*
>
>
>
> 2013/7/17 Matt Benson <[email protected]>
>
>> Great work, Romain!
>>
>> Matt
>>
>>
>> On Wed, Jul 17, 2013 at 10:34 AM, Romain Manni-Bucau
>> <[email protected]>wrote:
>>
>> > Hi,
>> >
>> > FYI with some hacks (EJB hacks (@Resource) + cdi interceptor hacks
>> (adding
>> > beans.xml with needed interceptors instead of using discovery) + jndi
>> hack
>> > (lookup)) we get now on this branch:
>> >
>> > *Tests run: 758, Failures: 0, Errors: 0, Skipped: 0*
>> > *
>> > *
>> > PS: it needs OWB trunk built with cdi-1.1 profile
>> >
>> > *Romain Manni-Bucau*
>> > *Twitter: @rmannibucau <https://twitter.com/rmannibucau>*
>> > *Blog: **http://rmannibucau.wordpress.com/*<
>> > http://rmannibucau.wordpress.com/>
>> > *LinkedIn: **http://fr.linkedin.com/in/rmannibucau*
>> > *Github: https://github.com/rmannibucau*
>> >
>> >
>> >
>> > 2013/7/1 Romain Manni-Bucau <[email protected]>
>> >
>> > > Hi guys,
>> > >
>> > > just created a branch to play with bean validation 1.1
>> > >
>> > > here it is https://svn.apache.org/repos/asf/bval/branches/bval-11/
>> > >
>> > > it doesn't fully pass TCKs ATM because we'd need cdi 1.1 impl (and it
>> > > would be great to use OWB ;) + some refactorings before going further
>> but
>> > > it is a start.
>> > >
>> > > *Romain Manni-Bucau*
>> > > *Twitter: @rmannibucau <https://twitter.com/rmannibucau>*
>> > > *Blog: **http://rmannibucau.wordpress.com/*<
>> > http://rmannibucau.wordpress.com/>
>> > > *LinkedIn: **http://fr.linkedin.com/in/rmannibucau*
>> > > *Github: https://github.com/rmannibucau*
>> > >
>> > >
>> >
>>
>
>

Reply via email to