I don't think the missing headers for 2 files mandates a respin of this RC . It is not really a case of 'incompatible' license or code that shouldn't be shipped. We have a top level license file that covers the entire project, including these files. IMO, We should fix it if there is a new RC for some other reason. But this alone doesn't seem to make new RC necessary.
Sushanth, Can you please reconsider your -1 ? On Fri, Jun 17, 2016 at 3:06 PM, Sushanth Sowmyan <khorg...@gmail.com> wrote: > -1, terribly sorry I didn't check for this earlier, but the RAT check > fails for this. > > If you run mvn apache-rat:check , then you see the following issue: > > Unapproved licenses: > > > /Users/sush/t/rel/apache-hive-2.1.0-src/common/src/java/org/apache/hive/common/util/DateParser.java > > /Users/sush/t/rel/apache-hive-2.1.0-src/common/src/test/org/apache/hive/common/util/TestDateParser.java > > Basically, these two files are missing the apache license header. We > need to add them in. > > All other things are good, though. It has the oracle fix I asked for > in RC2, md5s and signatures check out, compilation works on source > package, and I'm able to run the hive binary from the binary package. > I also tried a number of tests, and I've run a rat test on the release > > On Thu, Jun 16, 2016 at 6:02 PM, Jesus Camacho Rodriguez > <jcamachorodrig...@hortonworks.com> wrote: >> Apache Hive 2.1.0 Release Candidate 3 is available here: >> >> http://people.apache.org/~jcamacho/hive-2.1.0-rc3 >> >> Maven artifacts are available here: >> >> https://repository.apache.org/content/repositories/orgapachehive-1057/ >> >> Source tag for RC3 is at: >> https://github.com/apache/hive/releases/tag/release-2.1.0-rc3 >> >> >> Voting will conclude in 72 hours. >> >> Hive PMC Members: Please test and vote. >> >> Thanks. >> >> >> >>
