+1 Verified signature, checksum. No binary files in the distribution. Full build and RAT check passes. Unit tests passed.
thanks Prasad On Thu, Apr 9, 2015 at 10:38 AM, Patrick Hunt <[email protected]> wrote: > If they are generated files then it's ok. > > I'm +1 - the xsum/sig are valid, RAT runs clean (per the noted exception) > and the disclaimer, etc.. are correct. > > Patrick > > On Wed, Apr 8, 2015 at 10:37 AM, Prasad Mujumdar <[email protected]> > wrote: > > > Hi Patrick, > > > > Thanks for the feedback! > > > > Apache Sentry build invokes RAT check by default and these scripts > scripts > > are excluded [1] from the check. The .sql files are used for creating > > schema for Sentry metadata. Some of these are generated by native DB > export > > tools and others by cherry-picking parts of the top level script (for > > upgrades). In fact we did have some of these files in the last 1.4.0 > > release as well. There are other Apache projects like Apache Hive, which > > includes similar SQL files without license header [2]. > > > > Given that the SQLs are generated using external tools and there's > already > > a precedence of using such files without a license header, do you still > > think we should add licenses in Sentry ? > > > > thanks > > Prasad > > > > [1] - > > > https://github.com/apache/incubator-sentry/blob/branch-1.5.0/pom.xml#L710 > > [2] - > > > https://github.com/apache/hive/tree/trunk/metastore/scripts/upgrade/mysql > > > > > > On Wed, Apr 8, 2015 at 9:56 AM, Patrick Hunt <[email protected]> wrote: > > > > > I'm -1 at the moment - there are a number of .sql files > > > in ./sentry-provider/sentry-provider-db/src/main/resources that are > > missing > > > license headers. > > > > > > Patrick > > > > > > On Wed, Apr 8, 2015 at 1:31 AM, gqshen <[email protected]> wrote: > > > > > > > This is the incubator release of Apache Sentry, version > > 1.5.0-incubating. > > > > > > > > The list of fixed issues, added features and improvements can be > found > > > > here: > > > > > > > > > > > > > > https://git-wip-us.apache.org/repos/asf?p=incubator-sentry.git;a=blob;f=CHANGELOG.txt;h=5d96434653aa292dfd025c7baa876013e95a5a15;hb=refs/heads/branch-1.5.0 > > > > > > > > Source files : http://people.apache.org/~gqshen/sentry-1.5.0-rc0/ > > > > > > > > Tag to be voted on (release 1.5.0-rc0/SHA: > > > > 589ea2c36f7bbec2affbddf10c3320178fdec9bf): > > > > > > > > > > > > > > https://git-wip-us.apache.org/repos/asf?p=incubator-sentry.git;a=commit;h=589ea2c36f7bbec2affbddf10c3320178fdec9bf > > > > Sentry's KEYS containing the PGP key we used to sign the release: > > > > http://www.apache.org/dist/incubator/sentry/KEYS > > > > > > > > Note that this is a source only release and we are voting on the > > source: > > > > tag=release-1.5.0-rc0, SHA=589ea2c36f7bbec2affbddf10c3320178fdec9bf > > > > > > > > Vote will be opened for 72 hours. > > > > > > > > [ ] +1 approve > > > > [ ] +0 no opinion > > > > [ ] -1 disapprove (and reason why) > > > > > > > > Thank you for your prompt votes. > > > > > > > > Respectfully, > > > > > > > > Sentry 1.5.0 Release Manager (Guoquan Shen) > > > > > > > > > >
