zentol commented on a change in pull request #17458: URL: https://github.com/apache/flink/pull/17458#discussion_r727364094
########## 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 ``` +You can obviously control all skips manually too: +```bash +mvn clean install -DskipTests -Dskip.npm=true -Dcheckstyle.skip=true -Drat.skip=true -Dscalastyle.skip=true -Denforcer.skip=true -Dmaven.javadoc.skip=true -Djapicmp.skip=true -T 1C Review comment: I don't think we should be documenting the "manual" way in the first place. People who don't know maven should just stick to the `fast` profile a) for readability and b) to have a hook that works across all current and future Flink versions (e.g., they automatically reap the benefits if we add more stuff to the profile). -- 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
