zentol commented on a change in pull request #17458: URL: https://github.com/apache/flink/pull/17458#discussion_r727361542
########## 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: Its really not because this documentation is aimed at people who _do not know maven_, and putting them into a situation where the build fails for no reason is just not acceptable. -- 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: issues-unsubscr...@flink.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
