You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@bval.apache.org by "Mark Struberg (JIRA)" <ji...@apache.org> on 2010/03/17 11:22:27 UTC
[jira] Created: (BVAL-1) split usable validations like @Emain from
our impl
split usable validations like @Emain from our impl
--------------------------------------------------
Key: BVAL-1
URL: https://issues.apache.org/jira/browse/BVAL-1
Project: BeanValidation
Issue Type: Improvement
Reporter: Mark Struberg
Assignee: Mark Struberg
Currently there are a few very useful Validations included in the impl.
Since they are in fact independent of the actual JSR-303 implementation, we should split them out into two own modules
* validators-api which contains only the annotations and public stuff
* validators-impl which contains the implementation stuff which are not needed at compile time (always used with <scope>runtime</scope>)
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (BVAL-1) split usable validations like @Email
from our impl
Posted by "Mark Struberg (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/BVAL-1?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12870105#action_12870105 ]
Mark Struberg commented on BVAL-1:
----------------------------------
No no, no big reorg :)
It was only meant as a 'cleanup task'. I'm just not 100% sure if we can split the additional validator implementations so cleanly, or if they heavily reuse code from the core constraints. So maybe we have to 'dupe' some classes into this extensions-impl.jar for portability reasons.
> split usable validations like @Email from our impl
> --------------------------------------------------
>
> Key: BVAL-1
> URL: https://issues.apache.org/jira/browse/BVAL-1
> Project: BeanValidation
> Issue Type: Improvement
> Reporter: Mark Struberg
> Assignee: Mark Struberg
>
> Currently there are a few very useful Validations included in the impl.
> Since they are in fact independent of the actual JSR-303 implementation, we should split them out into two own modules
> * validators-api which contains only the annotations and public stuff
> * validators-impl which contains the implementation stuff which are not needed at compile time (always used with <scope>runtime</scope>)
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (BVAL-1) split usable validations like @Emain
from our impl
Posted by "Donald Woods (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/BVAL-1?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12870082#action_12870082 ]
Donald Woods commented on BVAL-1:
---------------------------------
Not sure I follow what you're proposing....
What exactly do we gain by splitting our implementation into a api and impl? There is a spec api that defines the API for anyone wanting to provide an impl, so doesn't that fit your idea of api vs. impl? Is there some OSGi specific packaging/best practices that you're wanting to follow?
>From a recent email thread about moving constraints - The spec defines constraints that all implementations must provide, but we can provide additional ones. Why would we move these out?
> split usable validations like @Emain from our impl
> --------------------------------------------------
>
> Key: BVAL-1
> URL: https://issues.apache.org/jira/browse/BVAL-1
> Project: BeanValidation
> Issue Type: Improvement
> Reporter: Mark Struberg
> Assignee: Mark Struberg
>
> Currently there are a few very useful Validations included in the impl.
> Since they are in fact independent of the actual JSR-303 implementation, we should split them out into two own modules
> * validators-api which contains only the annotations and public stuff
> * validators-impl which contains the implementation stuff which are not needed at compile time (always used with <scope>runtime</scope>)
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (BVAL-1) split usable validations like @Email
from our impl
Posted by "Roman Stumm (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/BVAL-1?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12880081#action_12880081 ]
Roman Stumm commented on BVAL-1:
--------------------------------
+1 from me
The ONLY constraint validator implementation, that uses some code from the core module is EmailValidator.
> split usable validations like @Email from our impl
> --------------------------------------------------
>
> Key: BVAL-1
> URL: https://issues.apache.org/jira/browse/BVAL-1
> Project: BeanValidation
> Issue Type: Improvement
> Reporter: Mark Struberg
> Assignee: Mark Struberg
>
> Currently there are a few very useful Validations included in the impl.
> Since they are in fact independent of the actual JSR-303 implementation, we should split them out into two own modules
> * validators-api which contains only the annotations and public stuff
> * validators-impl which contains the implementation stuff which are not needed at compile time (always used with <scope>runtime</scope>)
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (BVAL-1) split usable validations like @Email
from our impl
Posted by "Mark Struberg (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/BVAL-1?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12880083#action_12880083 ]
Mark Struberg commented on BVAL-1:
----------------------------------
remaining question is whether we like to do this in 1 jar or if we split API and impl into 2 jars.
I've only glimpsed over the code and it looks to me that 1 single constraints.jar would be enough, since there are not so many impl classes needed actually.
> split usable validations like @Email from our impl
> --------------------------------------------------
>
> Key: BVAL-1
> URL: https://issues.apache.org/jira/browse/BVAL-1
> Project: BeanValidation
> Issue Type: Improvement
> Reporter: Mark Struberg
> Assignee: Mark Struberg
>
> Currently there are a few very useful Validations included in the impl.
> Since they are in fact independent of the actual JSR-303 implementation, we should split them out into two own modules
> * validators-api which contains only the annotations and public stuff
> * validators-impl which contains the implementation stuff which are not needed at compile time (always used with <scope>runtime</scope>)
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (BVAL-1) split usable validations like @Emain
from our impl
Posted by "Mark Struberg (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/BVAL-1?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12870098#action_12870098 ]
Mark Struberg commented on BVAL-1:
----------------------------------
> Why would we move these out?
because the standard JSR-303 API will most probably be provided by the container (e.g. geronimo). So any non-standard validation must not be part of this package.
Because, if you code your application and like to use the @Email (which is not part of the JSR-303 standard API afaik), then you would end up with not being portable.
But if we split this additional functionality into an own api/impl jar (or a single 'extensions-impl.jar') then we would be perfectly portable. I assume that this 'extension-impl' only uses portable JSR-303 functions and will also run on other 303 implementations like Hibernate Validator.
> split usable validations like @Emain from our impl
> --------------------------------------------------
>
> Key: BVAL-1
> URL: https://issues.apache.org/jira/browse/BVAL-1
> Project: BeanValidation
> Issue Type: Improvement
> Reporter: Mark Struberg
> Assignee: Mark Struberg
>
> Currently there are a few very useful Validations included in the impl.
> Since they are in fact independent of the actual JSR-303 implementation, we should split them out into two own modules
> * validators-api which contains only the annotations and public stuff
> * validators-impl which contains the implementation stuff which are not needed at compile time (always used with <scope>runtime</scope>)
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (BVAL-1) split usable validations like @Email
from our impl
Posted by "Roman Stumm (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/BVAL-1?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12880085#action_12880085 ]
Roman Stumm commented on BVAL-1:
--------------------------------
My opinion: into 1 single jar, because
a) jar is small and users would most probably always need both if we would split
b) we already have several jars, better less than more.
Size of jar is less important than transitive dependencies, and there are no extra dependencies in the single jar anyway.
> split usable validations like @Email from our impl
> --------------------------------------------------
>
> Key: BVAL-1
> URL: https://issues.apache.org/jira/browse/BVAL-1
> Project: BeanValidation
> Issue Type: Improvement
> Reporter: Mark Struberg
> Assignee: Mark Struberg
>
> Currently there are a few very useful Validations included in the impl.
> Since they are in fact independent of the actual JSR-303 implementation, we should split them out into two own modules
> * validators-api which contains only the annotations and public stuff
> * validators-impl which contains the implementation stuff which are not needed at compile time (always used with <scope>runtime</scope>)
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (BVAL-1) split usable validations like @Email from
our impl
Posted by "Mark Struberg (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/BVAL-1?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Mark Struberg updated BVAL-1:
-----------------------------
Summary: split usable validations like @Email from our impl (was: split usable validations like @Emain from our impl)
> split usable validations like @Email from our impl
> --------------------------------------------------
>
> Key: BVAL-1
> URL: https://issues.apache.org/jira/browse/BVAL-1
> Project: BeanValidation
> Issue Type: Improvement
> Reporter: Mark Struberg
> Assignee: Mark Struberg
>
> Currently there are a few very useful Validations included in the impl.
> Since they are in fact independent of the actual JSR-303 implementation, we should split them out into two own modules
> * validators-api which contains only the annotations and public stuff
> * validators-impl which contains the implementation stuff which are not needed at compile time (always used with <scope>runtime</scope>)
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (BVAL-1) split usable validations like @Email
from our impl
Posted by "Donald Woods (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/BVAL-1?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12870102#action_12870102 ]
Donald Woods commented on BVAL-1:
---------------------------------
OK, that makes sense. Any additional constraints would be defined in separate api/impl jars.
Thought you were alluding to some grander reorg. :-)
> split usable validations like @Email from our impl
> --------------------------------------------------
>
> Key: BVAL-1
> URL: https://issues.apache.org/jira/browse/BVAL-1
> Project: BeanValidation
> Issue Type: Improvement
> Reporter: Mark Struberg
> Assignee: Mark Struberg
>
> Currently there are a few very useful Validations included in the impl.
> Since they are in fact independent of the actual JSR-303 implementation, we should split them out into two own modules
> * validators-api which contains only the annotations and public stuff
> * validators-impl which contains the implementation stuff which are not needed at compile time (always used with <scope>runtime</scope>)
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.