Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Test: db 커넥션 테스트를 위한 브랜치입니다. #51

Merged
merged 1 commit into from
Dec 16, 2023

Conversation

dev-khg
Copy link
Contributor

@dev-khg dev-khg commented Dec 16, 2023

No description provided.

@dev-khg dev-khg merged commit 73b5828 into dev Dec 16, 2023
2 checks passed

spring.datasource.hikari.maximum-pool-size=5
spring.datasource.hikari.minimum-idle=5
spring.datasource.hikari.idleTimeout=60000 No newline at end of file

Choose a reason for hiding this comment

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

이 코드 패치는 스프링 부트 애플리케이션의 데이터베이스 설정을 수정하는 것으로 보입니다. 여기에는 MariaDB JDBC 드라이버 설정과 HikariCP 커넥션 풀의 최대 크기, 최소 유지 개수, 대기 시간 등이 포함됩니다.

버그 위험 및 개선 제안은 다음과 같습니다:

  1. 마지막 라인에 새 줄이 없습니다. 이것은 개발자들 사이에서 권장되는 습관입니다. 파일의 끝에 적절한 개행 문자를 추가해 주세요.

  2. 현재 코드는 최대 풀 크기를 5로 설정하고, 최소 유지 개수와 대기 시간 역시 5와 60000으로 각각 설정하고 있습니다. 이 값들은 애플리케이션에 맞게 조정되어야 합니다. 예를 들어, 예상 동시 접속 수 및 서버 자원 상황에 따라 해당 값을 조정하여 최적의 성능을 얻을 수 있습니다.

  3. 다른 설정 옵션 (예: 최대 연결 시간, 유휴 커넥션 검사 등)도 고려할 가치가 있을 수 있으므로, 프로덕션 환경에 적합한 추가 설정을 고려해 보세요.

  4. 현재 코드는 데이터베이스의 URL, 사용자 이름, 암호 등의 정보가 빠져 있습니다. 이러한 정보는 보안상 중요하므로 필요한 경우 외부 설정 파일에 저장하거나 환경 변수를 사용하는 것이 좋습니다.

이외에도 코드 리뷰는 일반적으로 프로젝트의 특정 요구 사항과 설계 원칙을 고려하여 수행되어야 합니다. 하지만 여기에서 확인할 수 있는 내용은 위와 같습니다.

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