Skip to content

Bizgo 로그 추가#413

Merged
stemmmm merged 30 commits into
mainfrom
develop
Jan 14, 2026
Merged

Bizgo 로그 추가#413
stemmmm merged 30 commits into
mainfrom
develop

Conversation

@stemmmm
Copy link
Copy Markdown
Contributor

@stemmmm stemmmm commented Jan 14, 2026

Summary by CodeRabbit

릴리스 노트

  • 성능 개선

    • 알림 조회 성능 최적화를 위한 데이터베이스 인덱스 추가
  • 버그 수정

    • 메시지 발송 API 오류 발생 시 상세한 오류 정보 로깅 기능 강화

✏️ Tip: You can customize this high-level summary in your review settings.

stemmmm and others added 30 commits December 12, 2025 12:23
- 인터뷰 질문, 미션, 데이팅 시험 관련 기본 데이터 마이그레이션 추가
- Flyway 설정 로직에 비활성화 처리 조건 추가
- Hibernate DDL 설정 방식 변경 및 스키마 생성 스크립트 설정 추가

#377
# Conflicts:
#	src/main/java/deepple/deepple/common/config/FlywayConfig.java
- k6 테스트 설정 파일 및 API 테스트 스크립트 추가
- 테스트 환경 변수 및 부하 프로파일 설정
- README에 k6 관련 문서 및 실행 방법 추가

#394
- 성능 테스트용 SQL 데이터 생성 스크립트 추가
- 회원, 소개, 좋아요, 매칭, 알림 데이터 포함
- 데이터 검증 및 README 파일 추가

#394
- FCM 제외 테스트 로직 `sendWithoutPush` 추가
- k6 부하 테스트 스크립트 구현
- 테스트용 SQL 데이터 초기화/생성 로직 추가

#394
* [Refac] : When Match Request or Response, Don't Check Primary Contatct Type

* [fix] : Change BaseURL

* [Fix] : Using Enum
* [Refac] : When Match Request or Response, Don't Check Primary Contatct Type

* [fix] : Change BaseURL

* [Fix] : Using Enum

* [Refac] Change BizgoRequestBody
* [Refac] : When Match Request or Response, Don't Check Primary Contatct Type

* [fix] : Change BaseURL

* [Fix] : Using Enum

* [Refac] Change BizgoRequestBody

* [Fix] : Use Record for serialization
* [Refac] : When Match Request or Response, Don't Check Primary Contatct Type

* [fix] : Change BaseURL

* [Fix] : Using Enum

* [Refac] Change BizgoRequestBody

* [Fix] : Use Record for serialization

* [Refac] : Add Logging for checking.
- 데이터 접근 레이어 분리 (Reader/Writer) 및 저장 로직 개선
- FCM 예외 처리 로깅 추가
- 알림 템플릿 정보 전송 방식 변경
- k6 부하 테스트 시 최대 VU 증가
- notification_preferences 테이블에 인덱스 추가
- 테스트 코드 리팩토링 및 명확한 실패 상태 로깅 적용

#394
- CircuitBreaker 상태 전환, 실패율 초과, 느린응답률 초과 이벤트에 대한 로그 등록
- 이벤트 리스너 등록 로직 추가 및 @slf4j 어노테이션 적용
- 알림 테이블에 복합 인덱스 추가 (receiverId, deletedAt, id)
- 데이터베이스 및 엔티티 매핑 수정

#394
- Bizgo API 응답에서 에러 상태 코드 및 본문 로깅 추가
- 예외 발생 시 디버깅 용이하도록 개선

#409
@stemmmm stemmmm self-assigned this Jan 14, 2026
@stemmmm stemmmm added the enhancement New feature or request label Jan 14, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Jan 14, 2026

Caution

Review failed

The pull request is closed.

Walkthrough

이 변경사항은 세 가지 개선을 포함합니다: (1) BizgoMessanger에서 Bizgo API 응답 에러 발생 시 HTTP 상태 코드와 응답 본문을 모두 로깅하는 기능 추가, (2) Notification 엔티티의 @Table 어노테이션에 receiverId, deletedAt, id 컬럼 기반의 데이터베이스 인덱스 추가, (3) notifications 테이블에 해당 복합 인덱스를 생성하는 SQL 마이그레이션 추가입니다.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch develop


📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 7c7b1dc and 7b8c6c1.

📒 Files selected for processing (3)
  • src/main/java/deepple/deepple/member/command/infra/member/sms/BizgoMessanger.java
  • src/main/java/deepple/deepple/notification/command/domain/Notification.java
  • src/main/resources/db/migration/V7__add_index_to_notifications.sql

✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.


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 and usage tips.

@stemmmm stemmmm merged commit 1b7c9d1 into main Jan 14, 2026
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants