sqoop-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hudson (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (SQOOP-2283) Support usage of --exec and --password-alias
Date Wed, 08 Apr 2015 03:25:12 GMT

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

Hudson commented on SQOOP-2283:
-------------------------------

SUCCESS: Integrated in Sqoop-ant-jdk-1.6-hadoop23 #1183 (See [https://builds.apache.org/job/Sqoop-ant-jdk-1.6-hadoop23/1183/])
SQOOP-2283: Support usage of --exec and --password-alias (cshapi: https://git-wip-us.apache.org/repos/asf?p=sqoop.git&a=commit&h=d32137f15c06827dea50b293f4d422481cb374cd)
* src/java/org/apache/sqoop/SqoopOptions.java


> Support usage of --exec and --password-alias
> --------------------------------------------
>
>                 Key: SQOOP-2283
>                 URL: https://issues.apache.org/jira/browse/SQOOP-2283
>             Project: Sqoop
>          Issue Type: Bug
>            Reporter: Abraham Elmahrek
>            Assignee: Abraham Elmahrek
>             Fix For: 1.4.7
>
>         Attachments: SQOOP-2283.patch
>
>
> Currently, passwords are being queried for even when --password-alias is provided during
job creation.
> https://github.com/apache/sqoop/blob/trunk/src/java/org/apache/sqoop/SqoopOptions.java#L750
> {code}
>   private void writePasswordProperty(Properties props) {
>     if (getPasswordFilePath() != null) { // short-circuit
>       putProperty(props, "db.password.file", getPasswordFilePath());
>       return;
>     }
>     if (this.getConf().getBoolean(
>       METASTORE_PASSWORD_KEY, METASTORE_PASSWORD_DEFAULT)) {
>       // If the user specifies, we may store the password in the metastore.
>       putProperty(props, "db.password", this.password);
>       putProperty(props, "db.require.password", "false");
>     } else if (this.password != null) {
>       // Otherwise, if the user has set a password, we just record
>       // a flag stating that the password will need to be reentered.
>       putProperty(props, "db.require.password", "true");
>     } else {
>       // No password saved or required.
>       putProperty(props, "db.require.password", "false");
>     }
>   }
> {code}
> We should be aware of `db.password.alias` and set `db.require.password` appropriately.



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

Mime
View raw message