Hi ming wen, thanks for reply, I wil start a new vote. Ming Wen <wenm...@apache.org> 于2021年3月29日周一 下午3:57写道:
> yes, looks good to me. > > Thanks, > Ming Wen, Apache APISIX PMC Chair > Twitter: _WenMing > > > 寒江雪 <yangz...@gmail.com> 于2021年3月29日周一 下午3:38写道: > > > Hi ming wen, > > Is it possible to exclude all the unit tests in our project source code > > when packaging the release tarball ? > > > > Ming Wen <wenm...@apache.org> 于2021年3月29日周一 上午10:59写道: > > > > > hello, mingyu, > > > These files can be in the GitHub repo, but should not be in the > released > > > source package. > > > You can remove the test-related files when packaging, this is also the > > > usual practice. > > > > > > Thanks, > > > Ming Wen, Apache APISIX PMC Chair > > > Twitter: _WenMing > > > > > > > > > 陈明雨 <morning...@163.com> 于2021年3月29日周一 上午10:48写道: > > > > > > > Hi kezhenxu94: > > > > > > > > > > > > Thanks for your apply. > > > > > > > > > > > > We have such a scenario, such as the need to test a file > decompression > > > > code in a unit test. > > > > I think a reasonable way is to directly provide some small zip files > > for > > > > the unit test to test the > > > > decompression logic without writing a set of compression code to > > generate > > > > these zip files at runtime. > > > > > > > > > > > > And in fact, we also refer to the practices of some top-level > projects, > > > > such as [1], [2]. > > > > > > > > > > > > In addition, I only found instructions on the Release that the main > > > > running code cannot contain binary, > > > > but the instructions for unit testing are rather vague, so we are not > > > > completely sure how to deal with these problems reasonably. > > > > If there are related links, could you please provide us for > reference? > > > > > > > > > > > > [1] > > > > > > > > > > https://github.com/apache/spark/tree/v3.1.1/sql/hive/src/test/resources/regression-test-SPARK-8489 > > > > [2] https://github.com/apache/impala/tree/3.4.0/testdata/tzdb > > > > > > > > > > > > > > > > > > > > -- > > > > > > > > 此致!Best Regards > > > > 陈明雨 Mingyu Chen > > > > > > > > Email: > > > > morning...@163.com; > > > > morningman....@gmail.com > > > > > > > > > > > > > > > > > > > > > > > > 在 2021-03-28 22:55:11,"kezhenxu94" <kezhenx...@apache.org> 写道: > > > > > > > > > > > > > > >> On Mar 28, 2021, at 18:40, 陈明雨 <morning...@163.com> wrote: > > > > >> > > > > >> Hi kezhenxu94: > > > > >> > > > > >> > > > > >> These files are for unit tests to test the java dynamic library > > > > loading, not some kind of "test tools". > > > > > > > > > >I have some further questions, where are these files come from, if > > their > > > > source codes are in the same repo, why not just build them from the > > > source > > > > codes before using them, if the files’ source codes are not in the > same > > > > repo, there a bunch of ways to depend on them, but including the > binary > > > > files is not the first choice, if worse, the files’ source codes are > > not > > > > accessible, the PMC has no way to review the content and it’s a > > > potential > > > > security problem. > > > > > > > > > >> These files are in the code repo for a long time and also > contained > > in > > > > our previous releases, and no one disagrees with it. > > > > >> So I believe it is ok for a source release. > > > > > > > > > >The previous releases including them does not mean it’s ok IMO, it > > might > > > > be overlooked , that’s one of the reasons why we need several > releases > > > > before graduation to give the IPMC more time to review the release > > > process > > > > as well as their contents. > > > > > > > > > >> > > > > >> > > > > >> > > > > >> -- > > > > >> > > > > >> 此致!Best Regards > > > > >> 陈明雨 Mingyu Chen > > > > >> > > > > >> Email: > > > > >> chenmin...@apache.org > > > > >> > > > > >> > > > > >> > > > > >> > > > > >> > > > > >>> 在 2021-03-27 22:53:49,"kezhenxu94@apache" <kezhenx...@apache.org > > > > > 写道: > > > > >>> Sorry my -1, I saw there are binary files in the source tar, > > although > > > > they’re test tools, but this is not allowed in the source release. > > > > >>> > > > > >>> ./fe/fe-core/src/test/resources/plugin_test/plugin_test.jar > > > > >>> > > > > > > > > > > ./fe/fe-core/src/test/resources/plugin_test/test_local_plugin/auditdemo.jar > > > > >>> > > > > >>>>> On Mar 26, 2021, at 12:36, 寒江雪 <yangz...@gmail.com> wrote: > > > > >>>> > > > > >>>> Hi all, > > > > >>>> > > > > >>>> Please review and vote on Apache Doris 0.14.0-incubating-rc03 > > > 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/r1d3728b02b4f7b56413d3bf334b94fbc9c8406bf2d78aa01fea60a77%40%3Cdev.doris.apache.org%3E > > > > >>>> > > > > >>>> The vote result email thread: > > > > >>>> > > > > > > > > > > https://lists.apache.org/thread.html/re1eb419d0eb4689d77e82727811facef08f3c2498f64ad2ea812d42a%40%3Cdev.doris.apache.org%3E > > > > >>>> > > > > >>>> The release candidate has been tagged in GitHub as 0.14.0-rc03, > > > > available > > > > >>>> here: > > > > >>>> > > https://github.com/apache/incubator-doris/releases/tag/0.14.0-rc03 > > > > >>>> > > > > >>>> 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-rc03/ > > > > >>>> > > > > >>>> 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 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 be install and start docker service, and then > > you > > > > could > > > > >>>> build Doris as 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 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-rc03/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 > > > > > > > > > > >--------------------------------------------------------------------- > > > > >To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org > > > > >For additional commands, e-mail: general-h...@incubator.apache.org > > > > > > > > > >