Skip to content

Conversation

@Gyuhyeok99
Copy link
Contributor

관련 이슈

작업 내용

불필요한 로그를 삭제합니다.

특이 사항

리뷰 요구사항 (선택)

@coderabbitai
Copy link

coderabbitai bot commented Nov 3, 2025

Caution

Review failed

The pull request is closed.

작업 개요

두 개의 로깅 관련 변경사항이 있습니다:

  1. UpdateViewCountService.java: updateViewCount 메서드 내에서 postId 계산 전과 조회수 업데이트 후 두 개의 로그 문장 제거
  2. UpdateViewCountScheduler.java: Lombok 기반 로깅 지원 제거 (import, @slf4j 애너테이션, log.info() 호출)

두 경우 모두 메서드의 트랜잭션 및 비동기 동작에는 영향이 없으며, 순수하게 로깅 출력만 제거된 변경입니다.

예상 코드 리뷰 노력

🎯 1 (경미함) | ⏱️ ~5분

  • 변경사항이 단순하고 반복적인 패턴 (로그 제거)
  • 공개 인터페이스나 동작 로직에는 변화 없음
  • 두 파일 모두 동일한 성격의 리팩토링으로 일관성 있음

제안 리뷰어

  • wibaek
  • whqtker
  • lsy1307
  • Hexeong
  • JAEHEE25
  • sukangpunch
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between be86fc5 and 36a68ab.

📒 Files selected for processing (2)
  • src/main/java/com/example/solidconnection/community/post/service/UpdateViewCountService.java (0 hunks)
  • src/main/java/com/example/solidconnection/scheduler/UpdateViewCountScheduler.java (0 hunks)

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.

Copy link

@JAEHEE25 JAEHEE25 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

@Gyuhyeok99 Gyuhyeok99 merged commit eaaf3b9 into solid-connection:develop Nov 3, 2025
1 of 2 checks passed
Gyuhyeok99 added a commit that referenced this pull request Nov 4, 2025
fix: PostRepository & CommentRepository의 일부 메서드의 정렬 순서 오류 해결 (#522) 
refactor: 이메일 유니크키 제거
refactor: 엔티티가 BaseEntity를 상속하도록 (#524) 
fix: BaseEntity 관련 마이그레이션 파일 수정 (#530) 
feat: 멘토 승격 api 구현 (#532) 
fix: 대학교 중복 오류 수정 (#510) 
refactor: 불필요한 로그 삭제 (#543)
fix: 운영환경 8081 포트 설정 추가 (#542) 
fix: dev환경 디비명 변경 (#546)
refactor: 불필요한 로그 삭제 (#547) 
refactor: 학기를 테이블로 관리하도록 변경 (#526) 
refactor: 모의지원 시 지원한 대학 정보 응답 추가 (#539)
@Gyuhyeok99 Gyuhyeok99 mentioned this pull request Nov 4, 2025
@Gyuhyeok99 Gyuhyeok99 deleted the refactor/541-delete-log branch November 9, 2025 09:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

refactor: 불필요한 로그 삭제

2 participants