Skip to content

Conversation

@mete0rfish
Copy link
Member

@mete0rfish mete0rfish commented Oct 24, 2025

User description

#️⃣ 이슈

none

🔎 작업 내용

현재 운영 서버에 연결된 PSQL 서버의 커넥션이 가득차는 문제가 발생.
과거 hikariCP의 maximum-pool-size를 PSQL의 최대 커넥션 크기를 넘지 않도록 수정했음.
그러나 DB 작업 후, 유휴 커넥션이 그대로 남아있는 문제가 발생.
따라서 Connection Pool의 유휴 시간과 최대 생명주기 시간을 설정하여 커넥션 누수가 발생하지 않도록 해결.

🤔 고민해볼 부분 & 코드 리뷰 희망사항

🧲 참고 자료 및 공유 사항

추후 트러블 슈팅 문서 첨부 예정


PR Type

Bug fix


Description

  • Fixes a connection leak issue in the production database server.

  • Configures HikariCP connection pool settings:

    • Sets leak-detection-threshold to 300000ms (5 minutes).
    • Sets connection-timeout to 300000ms (5 minutes).
    • Sets max-lifetime to 1800000ms (30 minutes).
  • Disables show_sql property.


Diagram Walkthrough

flowchart LR
  A["application.properties"] --> B("HikariCP Configuration");
  B -- "Set leak-detection-threshold" --> C("300000ms");
  B -- "Set connection-timeout" --> D("300000ms");
  B -- "Set max-lifetime" --> E("1800000ms");
  A --> F("Disable show_sql");
Loading

File Walkthrough

Relevant files
Configuration changes
application.properties
Configured HikariCP and disabled show_sql property             

server/src/main/resources/application.properties

  • Configured HikariCP connection pool settings to prevent connection
    leaks by setting leak-detection-threshold, connection-timeout, and
    max-lifetime.
  • Disabled the show_sql property to reduce logging overhead.
+9/-2     

@mete0rfish mete0rfish self-assigned this Oct 24, 2025
@mete0rfish mete0rfish added the 🚨 hotfix 긴급한 핫픽스 label Oct 24, 2025
@mete0rfish mete0rfish temporarily deployed to hotfix-connection-pool - backend PR #239 October 24, 2025 01:31 — with Render Destroyed
@mete0rfish
Copy link
Member Author

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

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

@mete0rfish
Copy link
Member Author

PR Code Suggestions ✨

No code suggestions found for the PR.

@mete0rfish mete0rfish merged commit edbf076 into main Oct 24, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants