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

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/10203/#review18553
-----------------------------------------------------------
This looks good to me with the exception of one thought. The client will be connecting to some RESTful resource, or at least Web like resource, ( IE: http://%(host)s:%(port)d/%(resource) ). Does it make sense to use idioms that reference a URI naming conventions?

+1
client/src/main/java/org/apache/sqoop/client/core/Constants.java
<https://reviews.apache.org/r/10203/#comment38880>

    SQOOP2_RESOURCE?
- Abraham Elmahrek
On March 30, 2013, 12:56 a.m., Jarek Cecho wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10203/
> -----------------------------------------------------------
>
> (Updated March 30, 2013, 12:56 a.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
>
>