Hey, I found some of the files lack the license headers, though some of which are for testing only, but since you have included them in the source tar, it’d be better to have license headers for them.
pinot-clients/pinot-jdbc-client/src/main/resources/java.sql.Driver pinot-connectors/pinot-spark-connector/src/main/resources/META-INF/services/org.apache.spark.sql.sources.DataSourceRegister pinot-core/src/test/resources/conf/pinot-broker.properties pinot-core/src/test/resources/conf/query-executor.properties pinot-core/src/test/resources/generate-test-data.py pinot-integration-tests/src/test/resources/compat-tests/kafka/topic1.cfg pinot-integration-tests/src/test/resources/index.html pinot-tools/src/main/resources/conf/sample_filesystem_benchmark.conf Also, I found that this file `docs/Makefile` originated from 3-rd party [1] and is modified in the release, but not mentioned in the LICENSE file. [1] https://github.com/apache/incubator-pinot/blob/release-0.7.0-rc/LICENSE > On Mar 23, 2021, at 00:25, Yupeng Fu <yupeng...@gmail.com> wrote: > > Felix and others, > > The git hash on the git tag is now the same as the one sent out in the > vote. Please check again > https://github.com/apache/incubator-pinot/commits/release-0.7.0 > > Sorry for the inconvenience and thanks for checking. > > Best, > > On Fri, Mar 19, 2021 at 3:45 PM Felix Cheung <felixche...@apache.org> wrote: > >> For some reason the git hash has changed in release git tag >> >> https://github.com/apache/incubator-pinot/commits/release-0.7.0 >> >> >> https://github.com/apache/incubator-pinot/commit/de0236c25727ffc2408c5bf55836bbfecf40499c >> >> >> >> >> On Fri, Mar 19, 2021 at 11:00 AM kishore g <g.kish...@gmail.com> wrote: >> >>> Jim/Felix and others >>> >>> Do we have your +1 to go ahead with the release. >>> >>> Thanks >>> Kishore G >>> >>> On Thu, Mar 18, 2021 at 9:32 PM kishore g <g.kish...@gmail.com> wrote: >>> >>>> That would be great. We will update the release runbook to make sure >> this >>>> doesn't happen again. >>>> >>>> Here is my +1 >>>> >>>> >>>> >>>> On Thu, Mar 18, 2021 at 11:03 AM Jim Jagielski <j...@jagunet.com> >> wrote: >>>> >>>>> I propose that we run the vote anyway, thus ensuring IPMC oversight >> and >>>>> (post)approval, and then update the release runbook to make sure this >>>>> doesn't happen again. Make sense? >>>>> >>>>>> On Mar 17, 2021, at 11:35 PM, Felix Cheung <felixche...@apache.org> >>>>> wrote: >>>>>> >>>>>> It is unclear the guidance on this but since it is impossible remove >>>>> from >>>>>> maven >>>>>> >>>>>> https://repository.apache.org/#nexus-search;classname~Pinot >>>>>> >>>>>> Once it is published, I’d suggest we update all the links and git >> hash >>>>> and >>>>>> still proceed to vote on this. If there is any issue we should vote >> on >>>>>> another release with fix ASAP. Otherwise hopefully IPMC can continue >>> to >>>>>> sign off on this release. >>>>>> >>>>>> >>>>>> >>>>>> On Wed, Mar 17, 2021 at 3:09 PM kishore g <g.kish...@gmail.com> >>> wrote: >>>>>> >>>>>>> Apologies for the mistake. We have done multiple releases until now >>> and >>>>>>> this is the first time we missed the step to get the approval from >>>>> IPMC was >>>>>>> missed before making the release. >>>>>>> >>>>>>> What is the right thing to do at this point? Should we cut another >>>>> release >>>>>>> or can we go ahead with this and ensure that this does not happen >>>>> again. >>>>>>> >>>>>>> thanks, >>>>>>> Kishore G >>>>>>> >>>>>>> On Wed, Mar 17, 2021 at 10:59 AM Felix Cheung < >>> felixche...@apache.org> >>>>>>> wrote: >>>>>>> >>>>>>>> Unfortunately this isn’t the right process for a podling release. >>>>>>>> >>>>>>>> As the mentor, I am sorry to see the release is not up to the >>> standard >>>>>>>> upheld by previous releases. >>>>>>>> >>>>>>>> >>>>>>>> On Wed, Mar 17, 2021 at 7:02 AM kezhenxu94@apache < >>>>> kezhenx...@apache.org >>>>>>>> >>>>>>>> wrote: >>>>>>>> >>>>>>>>> I think this release violates the Apache release policy because >> it >>>>>>> didn’t >>>>>>>>> get the approvals from IPMC but was released actually (in both >>>>> dist.a.o >>>>>>>> and >>>>>>>>> GitHub). >>>>>>>>> >>>>>>>>> ————————— >>>>>>>>> Zhenxu Ke (柯振旭) >>>>>>>>> GitHub @kezhenxu94 >>>>>>>>> >>>>>>>>>> On Mar 17, 2021, at 06:14, Yupeng Fu <yupeng...@gmail.com> >> wrote: >>>>>>>>>> >>>>>>>>>> Hi all, >>>>>>>>>> >>>>>>>>>> This is a call for vote to the release Apache Pinot (incubating) >>>>>>>> version >>>>>>>>>> 0.7.0. >>>>>>>>>> >>>>>>>>>> Apache Pinot (incubating) is a distributed columnar storage >> engine >>>>>>> that >>>>>>>>> can >>>>>>>>>> ingest data in realtime and serve analytical queries at low >>> latency. >>>>>>>>>> >>>>>>>>>> Pinot community has voted and approved this release. >>>>>>>>>> >>>>>>>>>> Vote thread: >>>>>>>>>> >>>>>>>>> >>>>>>>> >>>>>>> >>>>> >>> >> https://lists.apache.org/thread.html/r850cee1e6fc23804d161a31d465e30d5d12b7a4e1cc8035264ea91d6%40%3Cdev.pinot.apache.org%3E >>>>>>>>>> >>>>>>>>>> >>>>>>>>>> Result thread: >>>>>>>>>> >>>>>>>>> >>>>>>>> >>>>>>> >>>>> >>> >> https://lists.apache.org/thread.html/r80672ac39a6a6eb7f63c7dbbf595de2f7350cf75d2a7a598601da638%40%3Cdev.pinot.apache.org%3E >>>>>>>>>> >>>>>>>>>> The release candidate: >>>>>>>>>> >>>>>>>>> >>>>>>>> >>>>>>> >>>>> >>> >> https://dist.apache.org/repos/dist/dev/incubator/pinot/apache-pinot-incubating-0.7.0-rc1/ >>>>>>>>>> >>>>>>>>>> >>>>>>>>>> Git tag for this release: >>>>>>>>>> < >> https://github.com/apache/incubator-pinot/tree/release-0.7.0-rc0 >>>> >>>>>>>>>> >> https://github.com/apache/incubator-pinot/tree/release-0.7.0-rc1 >>>>>>>>>> >>>>>>>>>> Git hash for this release: >>>>>>>>>> 695ca390a8080409b3b76f250f2315b81b86b362 >>>>>>>>>> >>>>>>>>>> The artifacts have been signed with key: F41AB2B856C6C29D, which >>> can >>>>>>> be >>>>>>>>>> found in the following KEYS file. >>>>>>>>>> https://dist.apache.org/repos/dist/release/incubator/pinot/KEYS >>>>>>>>>> >>>>>>>>>> Release notes: >>>>>>>>>> >>>>>>>> >>>>> >>> https://github.com/apache/incubator-pinot/releases/tag/release-0.7.0-rc1 >>>>>>>>>> >>>>>>>>>> Staging repository: >>>>>>>>>> < >>>>>>>> >>>>> >> https://repository.apache.org/content/repositories/orgapachepinot-1023> >>>>>>>>>> >>>>>>> >>> https://repository.apache.org/content/repositories/orgapachepinot-1024 >>>>>>>>>> >>>>>>>>>> Documentation on verifying a release candidate: >>>>>>>>>> >>>>>>>>> >>>>>>>> >>>>>>> >>>>> >>> >> https://cwiki.apache.org/confluence/display/PINOT/Validating+a+release+candidate >>>>>>>>>> >>>>>>>>>> >>>>>>>>>> The vote will be open for at least 72 hours or until the >> necessary >>>>>>>>> number of >>>>>>>>>> votes are reached. >>>>>>>>>> >>>>>>>>>> Please vote accordingly, >>>>>>>>>> >>>>>>>>>> [ ] +1 approve >>>>>>>>>> [ ] +0 no opinion >>>>>>>>>> [ ] -1 disapprove with the reason >>>>>>>>>> >>>>>>>>>> Thanks, >>>>>>>>>> Apache Pinot (incubating) team >>>>>>>>> >>>>>>>>> >>>>>>>>> >>>>>>>>> >>> --------------------------------------------------------------------- >>>>>>>>> To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org >>>>>>>>> For additional commands, e-mail: >> general-h...@incubator.apache.org >>>>>>>>> >>>>>>>>> >>>>>>>> >>>>>>> >>>>> >>>>> >>>>> --------------------------------------------------------------------- >>>>> To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org >>>>> For additional commands, e-mail: general-h...@incubator.apache.org >>>>> >>>>> >>> >> > > > -- > --Yupeng ————————— Zhenxu Ke (柯振旭) GitHub @kezhenxu94 --------------------------------------------------------------------- To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org For additional commands, e-mail: general-h...@incubator.apache.org
