Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Threaded View
Sqoop, mail # dev - Review Request: SQOOP-972 Sqoop2: Load server URL from environment in shell


Copy link to this message
-
Re: Review Request: SQOOP-972 Sqoop2: Load server URL from environment in shell
Abraham Elmahrek 2013-04-10, 00:37

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/10203/#review18915
-----------------------------------------------------------

Ship it!
Ship It!

- Abraham Elmahrek
On March 30, 2013, 8:55 p.m., Jarek Cecho wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10203/
> -----------------------------------------------------------
>
> (Updated March 30, 2013, 8:55 p.m.)
>
>
> Review request for Sqoop.
>
>
> Description
> -------
>
> I've created environmental variables that will be fetched for Sqoop2 server default hostname (port, ...). Please note that resource file (if it exists) will have precedence over the environmental variables in current implementation.
>
>
> This addresses bug SQOOP-972.
>     https://issues.apache.org/jira/browse/SQOOP-972
>
>
> Diffs
> -----
>
>   client/src/main/java/org/apache/sqoop/client/core/Constants.java cc956d0f9306a08ef9084e7b878f82e3e355d617
>   client/src/main/java/org/apache/sqoop/client/shell/ShellEnvironment.java 25ae364641c7a03e8394ac14991370fc6f810745
>
> Diff: https://reviews.apache.org/r/10203/diff/
>
>
> Testing
> -------
>
> I've done manual testing as we currently do not have tests for shell component.
>
>
> Thanks,
>
> Jarek Cecho
>
>