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)