JingGe commented on a change in pull request #17458: URL: https://github.com/apache/flink/pull/17458#discussion_r731899795
########## File path: docs/content/docs/flinkDev/building.md ########## @@ -52,11 +52,19 @@ mvn clean install -DskipTests This instructs [Maven](http://maven.apache.org) (`mvn`) to first remove all existing builds (`clean`) and then create a new Flink binary (`install`). -To speed up the build you can skip tests, QA plugins, and JavaDocs: +To speed up the build you can: +- skip unit tests, integration tests and use the build-in "fast" and "skip-webui-build" maven profile to skip QA, JavaDocs, and frontend plugins. +- active maven parallel build, e.g. "mvn -T 1C" means 1 thread per cpu core. The maven parallel build may have deadlock(issue with the shade plugin). Please be aware of that and make sure you can live with it. +The build script will be: ```bash -mvn clean install -DskipTests -Dfast +mvn clean install -DskipTests -Dfast -Pskip-webui-build -T 1C ``` +The fast build profile will observably reduce the build time. You can also control each skip manually without using profiles, e.g. skip part of or more than the skip setup defined by the profiles, but it is recommended to stick to the fast and skip-webui-build profile, because: +- the maven build script has better readability. +- the profiles will be the abstraction that works across all current and future Flink versions. +- developer will automatically reap the benefits without extra effort if more stuff has been added to the profile. + Review comment: agree -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
