JingGe commented on a change in pull request #17458: URL: https://github.com/apache/flink/pull/17458#discussion_r727793607
########## 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: +- use the build-in "fast" maven profile which will skip unit tests, integration tests, QA plugins, and JavaDocs. +- active maven parallel build, e.g. "mvn -T 1C" means 1 thread per cpu core. +The build script will be: ```bash -mvn clean install -DskipTests -Dfast +mvn clean install -DskipTests -Dfast -T 1C Review comment: Your consideration is great! I would suggest to point out the risk in the documentation and let user make the choice, since I did find any another place that is better to tell people who do know maven about this improvement hint and info about building Flink should be kept at one place for people with different background. -- 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]
