dimas-b commented on code in PR #2964: URL: https://github.com/apache/polaris/pull/2964#discussion_r2487239165
########## CONTRIBUTING.md: ########## @@ -119,12 +119,82 @@ Tips: * Keep in mind that the Git commit subject and message is going to be read by other people, potentially even after years. The Git commit subject and message will appear "as is" in release notes. * Make sure the subject and message are properly formatted and contains a concise description of the changes in way that someone who has no prior knowledge can understand the rationale of the change and the change itself. Remove information that's of no use for someone reading the Git commit log, for example single intermediate commit messages like `formatting` or `fix test`. -## Java version requirements +## Build Prerequisites and Required Tools -The Apache Polaris build currently requires Java 21 or later. There are a few tools that help you running the right Java version: +Apache Polaris (incubating) requires several tools to run a full build, including tests and documentation generation. -* [SDKMAN!](https://sdkman.io/) follow the installation instructions, then run `sdk list java` to see the available distributions and versions, then run `sdk install java <identifier from list>` using the identifier for the distribution and version (>= 21) of your choice. -* [jenv](https://www.jenv.be/) If on a Mac you can use jenv to set the appropriate SDK. +### Core Build Requirements + +These tools are required for basic building and testing: + +* **Git**: Required for cloning the repository and version control. + +* **Java 21+**: The build requires Java 21 or later. Suggested installers: + * [SDKMAN!](https://sdkman.io/) - Run `sdk list java` to see available distributions, then `sdk install java <identifier>` to install. + * [jEnv](https://www.jenv.be/) - You can also use jEnv to manage Java versions. + +* **Docker**: Required for integration tests and building container images. Review Comment: nit: `Docker or Podman`? `Podman` with its docker compatibility shims works fairly well in my local build. -- 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]
