Skip to content

chore(build): trim default reactor to flight-sql-jdbc-driver chain#12

Open
mateusaubin wants to merge 1 commit into
mainfrom
feat/COM-668/trim-build-jdbc
Open

chore(build): trim default reactor to flight-sql-jdbc-driver chain#12
mateusaubin wants to merge 1 commit into
mainfrom
feat/COM-668/trim-build-jdbc

Conversation

@mateusaubin

Copy link
Copy Markdown
Collaborator

Trims the default Maven reactor to only the modules required by arrow-flight-sql-jdbc-driver (bom, format, memory, vector, adapter/jdbc, flight). Remaining upstream modules move to an opt-in full-build profile (mvn -Pfull-build ...).

  • ~35% less CPU (~21% wall clock) per clean build
  • Both profiles verified green with clean package
  • Also switches git-commit-id to native git with tag support so builds work from git worktrees

Linear: COM-668

Default build now includes only the modules required by
arrow-flight-sql-jdbc-driver (bom, format, memory, vector,
adapter/jdbc, flight). The remaining upstream modules move to an
opt-in `full-build` profile (`mvn -Pfull-build ...`).

Cuts clean-build time by ~35% CPU (~21% wall clock).

Also switches git-commit-id to native git with tag support, so
builds work from git worktrees.

issue: https://linear.app/iomete/issue/COM-668

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR optimizes the default Maven reactor to focus on the arrow-flight-sql-jdbc-driver build chain, moving the remaining upstream modules into an opt-in full-build profile. It also adjusts git-commit-id-plugin to use native git with tag-based describe support to improve compatibility with git worktrees.

Changes:

  • Trim the root pom.xml default <modules> list to only the Flight SQL JDBC driver dependency chain, and introduce a full-build profile that restores the removed modules.
  • Remove flight-integration-tests from the flight reactor and build it only via the root full-build profile.
  • Update git-commit-id-plugin configuration to use native git and enable tag-based git describe.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
pom.xml Reduces the default reactor module set, adds full-build profile modules, and updates git commit metadata generation configuration.
flight/pom.xml Excludes flight-integration-tests from the default Flight reactor so it builds only under the root full-build profile.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@mateusaubin
mateusaubin marked this pull request as ready for review July 16, 2026 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants