Skip to content

Conversation

@seongjinyoon
Copy link
Contributor

Description:
Integrate JOOQ code generation into SBT build system

Changes:
Add JOOQ generation task using SBT source generators that run before compilation
Add JOOQ dependencies required for build time code generation

@Copilot Copilot AI review requested due to automatic review settings September 19, 2025 20:02
Copy link
Contributor

Copilot AI left a comment

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 integrates JOOQ code generation into the SBT build system to automate database access object generation. The implementation adds a custom SBT task that generates JOOQ classes from database schema before compilation.

  • Adds JOOQ-related dependencies to the build system for code generation
  • Implements a custom SBT task that reads JOOQ configuration and database settings to generate code
  • Integrates the generation task into the compilation pipeline using SBT source generators

Reviewed Changes

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

File Description
core/project/plugins.sbt Adds JOOQ codegen, Typesafe config, and PostgreSQL driver dependencies
core/dao/build.sbt Implements JOOQ code generation task and integrates it into SBT compilation

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Contributor

@aglinxinyuan aglinxinyuan left a comment

Choose a reason for hiding this comment

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

LGTM!

@aglinxinyuan aglinxinyuan enabled auto-merge (squash) September 19, 2025 23:50
@aglinxinyuan aglinxinyuan merged commit db4f039 into apache:main Sep 20, 2025
17 of 18 checks passed
@seongjinyoon seongjinyoon deleted the feat/jooq branch September 20, 2025 00:13
aglinxinyuan added a commit that referenced this pull request Sep 21, 2025
Since the JOOQ code generator was merged into the sbt compiler in PR
#3746, there’s no longer a need for a separate step for JOOQ code
generator in CI.
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