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) > > > > > >
