Skip to content

fix(ebean): default datasource autoCommit to true#17960

Merged
craig-rueda merged 1 commit into
masterfrom
craig/ebean-autocommit-default
Jun 19, 2026
Merged

fix(ebean): default datasource autoCommit to true#17960
craig-rueda merged 1 commit into
masterfrom
craig/ebean-autocommit-default

Conversation

@craig-rueda

Copy link
Copy Markdown
Contributor

Summary

  • Changes the default value of ebean.autoCommit from false to true in both application.yaml and LocalEbeanConfigFactory.java
  • Ebean's connection pool performs better with autoCommit enabled — avoids wrapping every read in an unnecessary transaction
  • Operators can still override via EBEAN_DATASOURCE_AUTOCOMMIT=false

Ebean's connection pool performs better with autoCommit enabled — it
avoids wrapping every single read in an unnecessary transaction.
Operators can still override via EBEAN_DATASOURCE_AUTOCOMMIT=false.
@github-actions github-actions Bot added the devops PR or Issue related to DataHub backend & deployment label Jun 18, 2026
@codecov

codecov Bot commented Jun 18, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@craig-rueda craig-rueda merged commit d598750 into master Jun 19, 2026
60 of 62 checks passed
@craig-rueda craig-rueda deleted the craig/ebean-autocommit-default branch June 19, 2026 02:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

devops PR or Issue related to DataHub backend & deployment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants