Thank you for driving the release Stanley! I’ve took a look and it looks 
generally fine to me. It seems that we’ve added a lot of dependencies that 
we’re currently shipping with the binary artifacts inside the lib/ directory. 
We have a good practice to enumerate all the jars that we’re shipping with 
their licenses in LICENSE.txt file

https://github.com/apache/sqoop/blob/trunk/LICENSE.txt

Might I suggest to add the extra jars to the LICENSE file? I did not verified 
every one of them, but I’ve noticed that at least parquet-* opencsv* xz* are 
missing there.

Jarcec

> On Mar 31, 2015, at 10:35 AM, Xu, Qian A <[email protected]> wrote:
> 
> Dear Sqoop Developers, Committers and PMC members,
> 
> This is the first release for Apache Sqoop, version 1.4.6.
> 
> *** Please cast your vote by [3 working days after sending] ***
> 
> The list of fixed issues:
> https://git-wip-us.apache.org/repos/asf?p=sqoop.git;a=blob;f=CHANGELOG.txt;h=d94bb9e8d23dc72a8ebd749e34ee1038ca7958a1;hb=e28452a84efcc2bd24f1eb867098e39f6efecdd5
> The tarball (*.tar.gz), signature (*.asc), checksum (*.md5),
> and test result (log/*.ant_test.log):
> http://people.apache.org/~gwenshap/sqoop-1.4.6-rc0/
> 
> 
> The tag to be voted on:
> 
> https://git-wip-us.apache.org/repos/asf?p=sqoop.git;a=shortlog;h=refs/heads/branch-1.4.6
> 
> 
> 
> The KEYS file:
> 
> https://dist.apache.org/repos/dist/release/sqoop/KEYS
> 
> 
> Thanks for Gwen to be release mentor.
> 
> --Stanley (Qian) Xu
> 

Reply via email to