Sound great! I will try to separate Python into a different profiles, thanks!
On Thu, Mar 17, 2022 at 9:30 PM kezhenxu94 <[email protected]> wrote: > > Sounds reasonable not to build everything by default. However in order to be > able to build the Python module with ease (for people who use it or for > developers to verify), we can introduce a maven profile to build it > conditionally, for example ./mvnw package -Ppython > > > > > On Mar 17, 2022, at 19:16, Jiajie Zhong <[email protected]> wrote: > > > > Hey guys, > > > > I start a thread to discuss should we remove build Python distribute when > > users > > build package with comamdn `mvn install package -Prelease`? > > > > For now, in dev branch, run commd `mvn install package -Prelease` will > > build Python > > distribute package locally, and it requires users have Python in their > > environment and > > command `python` work in terminal. I add it by PR[1] because I find > > other Apache project > > which have Python API, also released Python API distribute package > > with the main project, > > See issue-8343[2] for more details. > > > > However, some users have reported that it is not user friendly to ask > > to have Python in > > environment to get the binary packages. Specially some users have > > integrated the upgrade > > stepinto the CD tool, such as Jenkins. And they expect to get the > > latest binary packages > > with as few dependencies as possible. > > > > I think it make sense because most of users do not use Python API(at > > least for now). If I am > > the user who do not use Python API, I also do not want to add > > dependencies for building > > package. But we release Python binary package together with main > > distirbute, it looks more professional. > > > > So do your guys think we should remove Python dependencies in building > > process or keep it? > > please share your ideas in this thread thanks > > > > [1]: https://github.com/apache/dolphinscheduler/pull/8470 > > [2]: https://github.com/apache/dolphinscheduler/issues/8343 > > > > -- > > Best Wish > > — Jiajie -- Best Wish — Jiajie
