Mailing-List: contact cloudstack-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: cloudstack-dev@incubator.apache.org
Received-SPF: neutral (nike.apache.org: local policy)
References: 
 <BEAC6B533902CC4DA3D452E248D90613012DDF33DBD6@SJCPMAILBOX01.citrite.net>
 <CA+96GG5pH6sKA+Esuqyc16aeCOHUSh-ataLZOL+vhRV-N7fV=A@mail.gmail.com>
 <04D8F7AD-0DC4-4766-87E9-4BC96AE0D472@citrix.com>
From: Chip Childers <chip.childers@sungard.com>
In-Reply-To: <04D8F7AD-0DC4-4766-87E9-4BC96AE0D472@citrix.com>
Mime-Version: 1.0 (1.0)
Date: Thu, 13 Dec 2012 00:42:50 -0500
Message-ID: <519595925327085130@unknownmsgid>
Subject: Re: [DISCUSS] Api refactoring -- validation
To: "<cloudstack-dev@incubator.apache.org>"
 <cloudstack-dev@incubator.apache.org>
Content-Type: text/plain; charset=ISO-8859-1

Thanks Min!  I think the comment about validating the DB entity was
what confused me. Got it now.

- chip

Sent from my iPhone.

On Dec 13, 2012, at 12:39 AM, Min Chen <min.chen@citrix.com> wrote:

> I think that it is the latter case.
>
> -min
>
> Sent from my iPhone
>
> On Dec 12, 2012, at 9:32 PM, "Chip Childers" <chip.childers@sungard.com> wrote:
>
>> On Wed, Dec 12, 2012 at 6:16 PM, Fang Wang <fang.wang@citrix.com> wrote:
>>> Hi all,
>>>
>>> In our current API refactoring work, we plan to add a validation annotation to
>>> Validate DB entity, validate some parameter range,  etc.
>>> We plan to add the @validate annotation to API commands.
>>
>> This might be a silly question, but why the command classes and not
>> the Dao's?  Is this to validate the database inputs?  Or are we
>> talking about cleaning up the API input validation logic?
>>
>> Thanks!
>>
>>> IT is suggested we could use Hibernate Validator.  Anyone with experience with
>>> Hibernate validator, how we could deploy it here?
>>>
>>> 1. is there a version we should definitely use?
>>> 2. Which option is better to integrate into our maven? Add it as dependency or as a plugin?
>>> 3. What is the latest version working with maven?
>>>
>>> Thanks for any input,
>>> -Fang
>