You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sqoop.apache.org by Abraham Elmahrek <ab...@cloudera.com> on 2015/03/19 02:13:08 UTC

Review Request 32228: SQOOP-2230: Sqoop2: Kite connector can use URIBuilder

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

Review request for Sqoop.


Bugs: SQOOP-2230
    https://issues.apache.org/jira/browse/SQOOP-2230


Repository: sqoop-sqoop2


Description
-------

commit c211e671f67ef9a8dccb3ba96c8f691ca378f178
Author: Abraham Elmahrek <ab...@apache.org>
Date:   Wed Mar 18 18:07:48 2015 -0700

    SQOOP-2230: Sqoop2: Kite connector can use URIBuilder

:100644 100644 b111b5c... 883925b... M  connector/connector-kite/src/main/java/org/apache/sqoop/connector/kite/KiteDatasetExecutor.java
:100644 100644 d48e250... 1df1859... M  connector/connector-kite/src/main/java/org/apache/sqoop/connector/kite/configuration/ConfigUtil.java
:100644 100644 eed6ee0... 7173c18... M  connector/connector-kite/src/test/java/org/apache/sqoop/connector/kite/TestKiteExecutor.java


Diffs
-----

  connector/connector-kite/src/main/java/org/apache/sqoop/connector/kite/KiteDatasetExecutor.java b111b5c 
  connector/connector-kite/src/main/java/org/apache/sqoop/connector/kite/configuration/ConfigUtil.java d48e250 
  connector/connector-kite/src/test/java/org/apache/sqoop/connector/kite/TestKiteExecutor.java eed6ee0 

Diff: https://reviews.apache.org/r/32228/diff/


Testing
-------

mvn test and manual tests.


Thanks,

Abraham Elmahrek


Re: Review Request 32228: SQOOP-2230: Sqoop2: Kite connector can use URIBuilder

Posted by Jarek Cecho <ja...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/32228/#review76994
-----------------------------------------------------------

Ship it!


Ship It!

- Jarek Cecho


On March 19, 2015, 1:13 a.m., Abraham Elmahrek wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/32228/
> -----------------------------------------------------------
> 
> (Updated March 19, 2015, 1:13 a.m.)
> 
> 
> Review request for Sqoop.
> 
> 
> Bugs: SQOOP-2230
>     https://issues.apache.org/jira/browse/SQOOP-2230
> 
> 
> Repository: sqoop-sqoop2
> 
> 
> Description
> -------
> 
> commit c211e671f67ef9a8dccb3ba96c8f691ca378f178
> Author: Abraham Elmahrek <ab...@apache.org>
> Date:   Wed Mar 18 18:07:48 2015 -0700
> 
>     SQOOP-2230: Sqoop2: Kite connector can use URIBuilder
> 
> :100644 100644 b111b5c... 883925b... M  connector/connector-kite/src/main/java/org/apache/sqoop/connector/kite/KiteDatasetExecutor.java
> :100644 100644 d48e250... 1df1859... M  connector/connector-kite/src/main/java/org/apache/sqoop/connector/kite/configuration/ConfigUtil.java
> :100644 100644 eed6ee0... 7173c18... M  connector/connector-kite/src/test/java/org/apache/sqoop/connector/kite/TestKiteExecutor.java
> 
> 
> Diffs
> -----
> 
>   connector/connector-kite/src/main/java/org/apache/sqoop/connector/kite/KiteDatasetExecutor.java b111b5c 
>   connector/connector-kite/src/main/java/org/apache/sqoop/connector/kite/configuration/ConfigUtil.java d48e250 
>   connector/connector-kite/src/test/java/org/apache/sqoop/connector/kite/TestKiteExecutor.java eed6ee0 
> 
> Diff: https://reviews.apache.org/r/32228/diff/
> 
> 
> Testing
> -------
> 
> mvn test and manual tests.
> 
> 
> Thanks,
> 
> Abraham Elmahrek
> 
>