sqoop-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jarek Cecho" <jar...@apache.org>
Subject Re: Review Request: SQOOP-973 add --url option for 'set server', which enables specifying host, port and webapp in a more consistant way with other applications. For example, 'set server --url http://localhost:12000/sqoop'
Date Wed, 05 Jun 2013 00:27:01 GMT

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


Nice patch Mengwei! I do have one additional comment:


client/src/main/java/org/apache/sqoop/client/shell/SetServerFunction.java
<https://reviews.apache.org/r/11535/#comment44435>

    I'm wondering what will happen when user will specify both --webapp and --url at the same
time. Do you think that it would make sense to address this edge case?


Jarcec

- Jarek Cecho


On June 4, 2013, 11:22 p.m., Mengwei Ding wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/11535/
> -----------------------------------------------------------
> 
> (Updated June 4, 2013, 11:22 p.m.)
> 
> 
> Review request for Sqoop, Jarek Cecho, Hari Shreedharan, and Abraham Elmahrek.
> 
> 
> Description
> -------
> 
> commit 67ccd56d492ac4910c28376f7be0bcaad4e5cbdb
> Author: Mengwei Ding <mengwei.ding@cloudera.com>
> Date:   Wed May 29 18:04:53 2013 -0700
> 
>     SQOOP-973 add --url option for 'set server', which enables specifying host, port
and webapp in a more consistant way with other applications. For example, 'set server --url
http://localhost:12000/sqoop'
> 
> :100644 100644 979b892... 68d9740... M	client/src/main/java/org/apache/sqoop/client/core/Constants.java
> :100644 100644 1c85592... c2581c3... M	client/src/main/java/org/apache/sqoop/client/shell/SetServerFunction.java
> :100644 100644 5dbcef5... 9f9289e... M	client/src/main/java/org/apache/sqoop/client/shell/ShellEnvironment.java
> :100644 100644 e328f9a... 2e91630... M	client/src/main/resources/client-resource.properties
> :100644 100644 33419b9... 2434a96... M	docs/src/site/sphinx/CommandLineClient.rst
> 
> 
> This addresses bug SQOOP-973.
>     https://issues.apache.org/jira/browse/SQOOP-973
> 
> 
> Diffs
> -----
> 
>   client/src/main/java/org/apache/sqoop/client/core/Constants.java 979b892 
>   client/src/main/java/org/apache/sqoop/client/shell/SetServerFunction.java 1c85592 
>   client/src/main/java/org/apache/sqoop/client/shell/ShellEnvironment.java 5dbcef5 
>   client/src/main/resources/client-resource.properties e328f9a 
>   docs/src/site/sphinx/CommandLineClient.rst 33419b9 
> 
> Diff: https://reviews.apache.org/r/11535/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Mengwei Ding
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message