You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sqoop.apache.org by Szabolcs Vasas <va...@gmail.com> on 2018/07/27 10:05:40 UTC

Review Request 68023: Remove Kite dependency from the Gradle dependencies

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

Review request for Sqoop.


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


Repository: sqoop-trunk


Description
-------

Removed Kite from the Gradle dependencies, cleaned up some config files containing references to Kite.


Diffs
-----

  LICENSE.txt 48b2c3bc4 
  gradle.properties 07224905a 
  gradle/sqoop-package.gradle 076fed858 
  testdata/hcatalog/conf/hive-site.xml 4922abd57 


Diff: https://reviews.apache.org/r/68023/diff/1/


Testing
-------

Unit and third party tests.


Thanks,

Szabolcs Vasas


Re: Review Request 68023: Remove Kite dependency from the Gradle dependencies

Posted by Boglarka Egyed <bo...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/68023/#review206543
-----------------------------------------------------------


Ship it!




Gradle test and 3rd part test suite ran successfully. Thanks for this clean up!

- Boglarka Egyed


On July 27, 2018, 10:05 a.m., Szabolcs Vasas wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/68023/
> -----------------------------------------------------------
> 
> (Updated July 27, 2018, 10:05 a.m.)
> 
> 
> Review request for Sqoop.
> 
> 
> Bugs: SQOOP-3349
>     https://issues.apache.org/jira/browse/SQOOP-3349
> 
> 
> Repository: sqoop-trunk
> 
> 
> Description
> -------
> 
> Removed Kite from the Gradle dependencies, cleaned up some config files containing references to Kite.
> 
> 
> Diffs
> -----
> 
>   LICENSE.txt 48b2c3bc4 
>   gradle.properties 07224905a 
>   gradle/sqoop-package.gradle 076fed858 
>   testdata/hcatalog/conf/hive-site.xml 4922abd57 
> 
> 
> Diff: https://reviews.apache.org/r/68023/diff/1/
> 
> 
> Testing
> -------
> 
> Unit and third party tests.
> 
> 
> Thanks,
> 
> Szabolcs Vasas
> 
>


Re: Review Request 68023: Remove Kite dependency from the Gradle dependencies

Posted by Fero Szabo via Review Board <no...@reviews.apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/68023/#review206542
-----------------------------------------------------------


Ship it!




Lgtm. Unit and 3rd party are successful.

- Fero Szabo


On July 27, 2018, 10:05 a.m., Szabolcs Vasas wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/68023/
> -----------------------------------------------------------
> 
> (Updated July 27, 2018, 10:05 a.m.)
> 
> 
> Review request for Sqoop.
> 
> 
> Bugs: SQOOP-3349
>     https://issues.apache.org/jira/browse/SQOOP-3349
> 
> 
> Repository: sqoop-trunk
> 
> 
> Description
> -------
> 
> Removed Kite from the Gradle dependencies, cleaned up some config files containing references to Kite.
> 
> 
> Diffs
> -----
> 
>   LICENSE.txt 48b2c3bc4 
>   gradle.properties 07224905a 
>   gradle/sqoop-package.gradle 076fed858 
>   testdata/hcatalog/conf/hive-site.xml 4922abd57 
> 
> 
> Diff: https://reviews.apache.org/r/68023/diff/1/
> 
> 
> Testing
> -------
> 
> Unit and third party tests.
> 
> 
> Thanks,
> 
> Szabolcs Vasas
> 
>