[
https://issues.apache.org/jira/browse/SQOOP-1467?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14109291#comment-14109291
]
Jarek Jarcec Cecho commented on SQOOP-1467:
-------------------------------------------
I would consider fact that I did not need to alter any test case positive in
this case - it means that changing the inheritance is not breaking any existing
test :)
> Sqoop2: Make MAccountableEntity inherit from MValidatedElement rather then
> MPersistableEntity
> ---------------------------------------------------------------------------------------------
>
> Key: SQOOP-1467
> URL: https://issues.apache.org/jira/browse/SQOOP-1467
> Project: Sqoop
> Issue Type: Improvement
> Reporter: Jarek Jarcec Cecho
> Assignee: Jarek Jarcec Cecho
> Fix For: from/to
>
> Attachments: SQOOP-1467.patch
>
>
> This will unify and simplify our model classes as currently {{MConnection}}
> and {{MJob}} are inheriting from {{MAccountableEntity}} which means that they:
> * Do not have any direct validations (which are however introduced via
> SQOOP-1439)
> * Have to implement their own "name" concept instead of re-using
> {{MNamedElement}} implementation (class that {{MValidatedElement}} further
> inherits from)
--
This message was sent by Atlassian JIRA
(v6.2#6252)