OK. I will check and fix the license header In next release, this thar OK ?

kezhenxu94@apache <kezhenx...@apache.org> 于2021年4月12日周一 上午10:25写道:

>
> > On Apr 12, 2021, at 09:33, 寒江雪 <yangz...@gmail.com> wrote:
> >
> > Hi Zhenxu Ke:
> > Thanks for voting. The following files are not from 3rd-party, they are
> > written by ourselves:
> > 1. be/src/olap/rowset/alpha_rowset_writer.h
> > 2. be/src/olap/rowset/segment_v2/empty_segment_iterator.cpp
>
> If that’s the case, the license header in file
> be/src/olap/rowset/segment_v2/empty_segment_iterator.cpp is not complete,
> please check and compare with other files, add the missing lines into it.
>
> > I will fix the wrong path of
> > "fe/fe-common/src/main/java/org/apache/doris/common/jmockit"  in
> LICENSE.txt
> >
> > kezhenxu94@apache <kezhenx...@apache.org> 于2021年4月11日周日 下午9:47写道:
> >
> >> Hi, I’m +1, here are my check list
> >>
> >> - LICENSE, DISCLAIM and NOTICE files exist;
> >> - Release candidate tar is signed correctly, by "yang...@apache.org”;
> >> - Shasum of the tar is correct;
> >> - “incubating” exists in tar file name and documentation;
> >> - License headers exist in files if necessary.
> >>
> >> However, I found
> >>
> >>  - the following files from 3rd-party are not listed in LICENSE.txt:
> >>    1. be/src/olap/rowset/alpha_rowset_writer.h
> >>    2. be/src/olap/rowset/segment_v2/empty_segment_iterator.cpp
> >>
> >>  - the following files are listed (IMO) but with wrong file path:
> >>    1. Should be
> >> "fe/fe-common/src/main/java/org/apache/doris/common/jmockit”, but was
> >> listed as "fe/fe-common/src/test/java/org/apache/doris/common/jmockit”
> in
> >> LICENSE.txt
> >>
> >> The missing 3rd-party files in LICENSE.txt is not a blocker IMO because
> >> there are files from the same owner ("be/src/olap/lru_cache.h”,
> >> “be/src/olap/lru_cache.cpp”, "be/src/olap/skiplist.h”) which are listed.
> >>
> >> And the wrong file path is easily fixed in mind so I think you can
> proceed
> >> but remember to fix in the next release.
> >>
> >>
> >>> On Apr 7, 2021, at 13:36, 寒江雪 <yangz...@gmail.com> wrote:
> >>>
> >>> Hi all,
> >>>
> >>> Please review and vote on Apache Doris 0.14.0-incubating-rc04 release.
> >>>
> >>> Apache Doris is an MPP-based interactive SQL data warehousing for
> >> reporting
> >>> and analysis.
> >>>
> >>> The Apache Doris community has voted on and approved this release:
> >>>
> >>
> https://lists.apache.org/thread.html/rc95a58e64b926dc89b61e747fd11f8cb7bb173d95395fa26d02d2c00%40%3Cdev.doris.apache.org%3E
> >>>
> >>> The vote result email thread:
> >>>
> >>
> https://lists.apache.org/thread.html/r14fc4ffeb31046343570a226dd7870ca74b0f15fbad492b09f0bb69f%40%3Cdev.doris.apache.org%3E
> >>>
> >>> The release candidate has been tagged in GitHub as 0.14.0-rc04,
> available
> >>> here:
> >>> https://github.com/apache/incubator-doris/releases/tag/0.14.0-rc04
> >>>
> >>> Thanks to everyone who has contributed to this release, and there is a
> >>> simple release notes can be found here:
> >>> https://github.com/apache/incubator-doris/issues/5374
> >>>
> >>> The artifacts (source, signature and checksum) corresponding to this
> >> release
> >>> candidate can be found here:
> >>>
> https://dist.apache.org/repos/dist/dev/incubator/doris/0.14/0.14.0-rc04/
> >>>
> >>> This has been signed with PGP key 59F2ACED, corresponding to
> >>> yang...@apache.org.
> >>> KEYS file is available here:
> >>> https://dist.apache.org/repos/dist/dev/incubator/doris/KEYS
> >>> It is also listed here:
> >>> https://people.apache.org/keys/committer/yangzhg.asc
> >>>
> >>> The vote will be open for at least 72 hours.
> >>> [ ] +1 Approve the release
> >>> [ ] +0 No opinion
> >>> [ ] -1 Do not release this package because ...
> >>>
> >>> To verify and build, you can refer to the following wiki:
> >>>
> >>
> https://github.com/apache/incubator-doris/wiki/How-to-verify-Apache-Release
> >>> https://wiki.apache.org/incubator/Incubator+Release+Checklist
> >>>
> >>> You can also refer to instruction:
> >>>
> >>> Firstly, you must install and start docker service, and then you could
> >>> build Doris as the following steps:
> >>>
> >>> Step1: Pull the docker image with Doris building environment
> >>> $ docker pull apachedoris/doris-dev:build-env-1.2
> >>> You can check it by listing images, its IMAGE ID is d854b488afaf
> >>>
> >>> Step2: Run the Docker image
> >>> You can run the image directly:
> >>> $ docker run -it apachedoris/doris-dev:build-env1.2
> >>>
> >>> Step3: Download Doris source
> >>> Now you should in the docker environment, and you can download the
> Doris
> >>> source package.
> >>> (If you have downloaded the source and it is not in image, you can map
> >> its
> >>> path to image in Step2.)
> >>> $ wget
> >>>
> >>
> https://dist.apache.org/repos/dist/dev/incubator/doris/0.14/0.14.0-rc04/apache-doris-0.14.0-incubating-src.tar.gz
> >>>
> >>> Step4: Build Doris
> >>> Now you can decompress and enter Doris source path and build Doris.
> >>> $ tar -zxvf apache-doris-0.14.0-incubating-src.tar.gz
> >>> $ cd apache-doris-0.14.0-incubating-src
> >>> $ sh build.sh
> >>>
> >>> Best Regards,
> >>> Yang Zhengguo
> >>
> >>
> >>
> >> —————————
> >> Zhenxu Ke (柯振旭)
> >> GitHub @kezhenxu94
> >>
> >>
> >> ---------------------------------------------------------------------
> >> To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
> >> For additional commands, e-mail: general-h...@incubator.apache.org
> >>
> >>
>
> —————————
> Zhenxu Ke (柯振旭)
> GitHub @kezhenxu94
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
> For additional commands, e-mail: general-h...@incubator.apache.org
>
>

Reply via email to