[
https://issues.apache.org/jira/browse/SQOOP-2608?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14953512#comment-14953512
]
Sqoop QA bot commented on SQOOP-2608:
-------------------------------------
Testing file
[SQOOP-2608.patch|https://issues.apache.org/jira/secure/attachment/12765895/SQOOP-2608.patch]
against branch sqoop2 took 0:59:53.416783.
{color:green}Overall:{color} +1 all checks pass
{color:green}SUCCESS:{color} Clean was successful
{color:green}SUCCESS:{color} Patch applied correctly
{color:green}SUCCESS:{color} Patch add/modify test case
{color:green}SUCCESS:{color} License check passed
{color:green}SUCCESS:{color} Patch compiled
{color:green}SUCCESS:{color} All unit tests passed (executed 1436 tests)
{color:green}SUCCESS:{color} Test coverage did not decreased
([report|https://builds.apache.org/job/PreCommit-SQOOP-Build/1775/artifact/patch-process/cobertura_report.txt])
{color:green}SUCCESS:{color} No new findbugs warnings
([report|https://builds.apache.org/job/PreCommit-SQOOP-Build/1775/artifact/patch-process/findbugs_report.txt])
{color:green}SUCCESS:{color} All integration tests passed (executed 89 tests)
Console output is available
[here|https://builds.apache.org/job/PreCommit-SQOOP-Build/1775/console].
This message is automatically generated.
> Sqoop2: MNamedElement should inherit from MValidatedElement
> -----------------------------------------------------------
>
> Key: SQOOP-2608
> URL: https://issues.apache.org/jira/browse/SQOOP-2608
> Project: Sqoop
> Issue Type: Bug
> Affects Versions: 1.99.6
> Reporter: Abraham Fine
> Assignee: Abraham Fine
> Attachments: SQOOP-2608.patch
>
>
> Currently {{MValidatedElement}} inherits from {{MNamedElement}}. This does
> not reflect the actual configuration we are trying to model. Currently
> {{ConfigurationClass}} has validations but is not named. There are no named
> elements that cannot have validations, so it makes sense to swap this part of
> the inheritance tree.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
