Skip to content

Add database connection pooler#496

Merged
DropSnorz merged 1 commit into
v2from
chore/connection-pool
Jul 15, 2026
Merged

Add database connection pooler#496
DropSnorz merged 1 commit into
v2from
chore/connection-pool

Conversation

@DropSnorz

Copy link
Copy Markdown
Owner

Relates to #494

@DropSnorz DropSnorz self-assigned this Jul 15, 2026
@coderabbitai

coderabbitai Bot commented Jul 15, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 396746a1-ef46-4d15-8cc3-09ab24154a53

📥 Commits

Reviewing files that changed from the base of the PR and between 4fc40dd and af50d54.

📒 Files selected for processing (1)
  • owlplug-client/src/main/java/com/owlplug/OwlPlug.java

Walkthrough

The client datasource bean now uses HikariDataSource instead of DriverManagerDataSource, applies JDBC connection properties through Hikari setters, and configures minimum and maximum pool sizes.

Changes

Datasource pooling

Layer / File(s) Summary
Configure Hikari datasource
owlplug-client/src/main/java/com/owlplug/OwlPlug.java
Replaces DriverManagerDataSource with HikariDataSource, uses setJdbcUrl, username, and password configuration, and adds pool sizing settings.

Estimated code review effort: 1 (Trivial) | ~5 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main change: adding a database connection pooler.
Description check ✅ Passed The description is brief, but it is related to the referenced issue and the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/connection-pool

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@DropSnorz

Copy link
Copy Markdown
Owner Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Jul 15, 2026

Copy link
Copy Markdown
✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@DropSnorz DropSnorz added this to the 2.0.0 milestone Jul 15, 2026
@DropSnorz
DropSnorz merged commit 5073f75 into v2 Jul 15, 2026
1 of 3 checks passed
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.

1 participant