Contributing

Get Started

This is a Rust project, so rustup is a great place to start. It provides an easy way to manage your Rust installation and toolchains.

This is a pure Rust project, so only cargo is needed. Here are some common commands to get you started:

  • cargo check: Analyze the current package and report errors. This is a quick way to catch any obvious issues without a full compilation.
  • cargo fmt: Format the current code according to the Rust style guidelines. This helps maintain a consistent code style throughout the project.
  • cargo build: Compile the current package. This will build the project and generate executable binaries if applicable.
  • cargo clippy: Catch common mistakes and improve code quality. Clippy provides a set of lints that can help you write better Rust code.
  • cargo test: Run unit tests. This will execute all the tests defined in the project to ensure the functionality is correct.
  • cargo bench: Run benchmark tests. This is useful for measuring the performance of specific parts of the code.

Setting up the Development Environment

  1. Install Rust using rustup. Follow the instructions on the rustup website to install Rust on your system.
  2. Clone the repository to your local machine.
  3. Navigate to the project directory.

Making Changes

  1. Create a new branch for your changes. This helps keep your work separate from the main development branch and makes it easier to review and merge your changes.
  2. Make your changes and ensure that the code still compiles and passes all tests. Use the commands mentioned above to check for errors and run tests.
  3. Format your code using cargo fmt to ensure consistency with the project's code style.

Submitting Changes

  1. Once you are satisfied with your changes, push your branch to the remote repository.
  2. Open a pull request on the project's GitHub page. Provide a clear description of your changes and why they are necessary.
  3. Wait for reviews and address any feedback. Once the pull request is approved and merged, your changes will be part of the project.

AI-Assisted Contributions

Apache Paimon Rust has the following policy for AI-assisted pull requests:

  • The PR author should understand the core ideas behind the implementation end-to-end and be able to justify the design and code during review.
  • Call out unknowns and assumptions. It is acceptable not to fully understand some details of AI-generated code, but authors should identify these cases and point them out to reviewers. This allows reviewers to apply their knowledge of the codebase and evaluate potential concerns, such as correctness, concurrency, compatibility, or performance risks.

Why fully AI-generated PRs without understanding are not helpful

AI tools cannot reliably make complex changes to Apache Paimon Rust on their own, which is why we rely on pull requests and code review.

The purposes of code review are:

  1. Complete the intended task correctly.
  2. Share knowledge between authors and reviewers as a long-term investment in the project.

A fully AI-generated contribution without sufficient author understanding does not meet these purposes. Maintainers could use AI tools directly, while contributors acting only as a proxy gain little knowledge of the project.

Review capacity is limited, so large pull requests that appear to lack the required understanding may not be reviewed and may eventually be closed or redirected.

Better ways to contribute than an “AI dump”

Consider writing a high-quality issue with a clear problem statement and a minimal reproducible example. This often makes it easier for the community to investigate the problem and develop an appropriate solution.

Read the design docs

For a deeper understanding of the project, read the design documentation available on our Paimon Rust.

Thank you for contributing to this project! 😊