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 >
