You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sqoop.apache.org by "Dian Fu (JIRA)" <ji...@apache.org> on 2015/06/03 04:08:49 UTC

[jira] [Commented] (SQOOP-2388) SQOOP2: Remove duplicate code in ConfigUtils.fromConfigs()

    [ https://issues.apache.org/jira/browse/SQOOP-2388?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14570136#comment-14570136 ] 

Dian Fu commented on SQOOP-2388:
--------------------------------

Haven't created review board as the patch is very simple.

> SQOOP2: Remove duplicate code in ConfigUtils.fromConfigs()
> ----------------------------------------------------------
>
>                 Key: SQOOP-2388
>                 URL: https://issues.apache.org/jira/browse/SQOOP-2388
>             Project: Sqoop
>          Issue Type: Bug
>            Reporter: Dian Fu
>            Assignee: Dian Fu
>         Attachments: SQOOP-2388.1.patch
>
>
> The following code in {{ConfigUtils.fromConfigs()}} is duplicate with {{getFieldFromName()}}
> {code}
> Field configField;
>       try {
>         configField = klass.getDeclaredField(config.getName());
>       } catch (NoSuchFieldException e) {
>         throw new SqoopException(ModelError.MODEL_006,
>           "Missing field " + config.getName() + " on config class " + klass.getCanonicalName(), e);
>       }
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)