Skip to content

952: Lowered maximum pool size to 1 in app yml - #897

Merged
Arshadul-Monir merged 1 commit into
mainfrom
952
Apr 13, 2026
Merged

952: Lowered maximum pool size to 1 in app yml#897
Arshadul-Monir merged 1 commit into
mainfrom
952

Conversation

@Arshadul-Monir

@Arshadul-Monir Arshadul-Monir commented Apr 13, 2026

Copy link
Copy Markdown
Member

952

Description of changes

Added yml property to lower max connections down to just 1 in staging

Checklist before review

  • I have done a thorough self-review of the PR
  • Copilot has reviewed my latest changes, and all comments have been fixed and/or closed.
  • If I have made database changes, I have made sure I followed all the db repo rules listed in the wiki here. (check if no db changes)
  • All tests have passed
  • I have successfully deployed this PR to staging
  • I have done manual QA in both dev (and staging if possible) and attached screenshots below.

Screenshots

Dev

Staging

@github-actions

Copy link
Copy Markdown
Contributor

Available PR Commands

  • /ai - Triggers all AI review commands at once
  • /review - AI review of the PR changes
  • /describe - AI-powered description of the PR
  • /improve - AI-powered suggestions
  • /deploy - Deploy to staging

See: https://github.com/tahminator/codebloom/wiki/CI-Commands

@Arshadul-Monir

Copy link
Copy Markdown
Member Author

/deploy

@github-actions

Copy link
Copy Markdown
Contributor

Title

952: Lowered maximum pool size to 1 in app yml


PR Type

Enhancement


Description

  • Configured Hikari connection pool.

  • Set maximum pool size to 1.

  • Applied to staging environment.

  • Updated application-stg.yml.


File Walkthrough

Relevant files
Configuration changes
application-stg.yml
Configure Hikari connection pool for staging                         

src/main/resources/application-stg.yml

  • Added spring.datasource.hikari.maximum-pool-size property.
  • Set the maximum database connection pool size to 1.
  • Applied this configuration specifically to the staging environment.
+5/-0     

@github-actions

Copy link
Copy Markdown
Contributor

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ No major issues detected

Comment thread src/main/resources/application-stg.yml
@Arshadul-Monir
Arshadul-Monir merged commit 8b08e5d into main Apr 13, 2026
34 checks passed
@Arshadul-Monir
Arshadul-Monir deleted the 952 branch April 13, 2026 16:42
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