ㅇㅇ

[wlsh44]

No description provided.

[gemini-code-assist]

Code Review

This pull request introduces a Redis-based caching layer for the 'like' functionality to improve performance, including a batch job for synchronizing likes with the RDB. The review identified several critical performance issues, such as N+1 query problems in the feed service and batch job, as well as data consistency concerns in the like removal logic. Additionally, there were suggestions to improve code cleanliness by removing unused imports and commented-out code, and to use more robust null-checking mechanisms.

[gemini-code-assist]

배치 작업 중 루프 내에서 getCreatedAt(key)를 호출하면 매번 Redis에 개별적으로 접근하게 되어 성능 저하를 유발합니다. redisTemplate.opsForValue().multiGet(keys)를 사용하여 필요한 데이터를 한 번에 가져오는 방식으로 개선이 필요합니다.

[gemini-code-assist]

현재 로직은 캐시 삭제에 성공하면 DB 삭제를 수행하지 않습니다. 하지만 배치 작업이 이미 실행되어 데이터가 DB에 반영되었을 가능성이 있으므로, 캐시 존재 여부와 상관없이 DB에서도 삭제 명령을 수행해야 데이터 정합성을 유지할 수 있습니다. 또한 더 이상 사용되지 않는 isCacheDeleted 메서드도 함께 제거하는 것이 좋습니다.

    @Override
    public void removeByMemberIdAndPostId(Long memberId, Long postId) {
        String key = redisService.makeKey(RedisPrefix.LIKE_PUSH, memberId, postId);
        redisTemplate.delete(key);
        likeJpaRepository.deleteByMemberIdAndPostId(memberId, postId);
    }

[gemini-code-assist]

스트림 내부에서 isMemberLikePost와 likeRepository.countByPostId를 호출하면 포스트 개수만큼 추가적인 Redis/DB 조회가 발생하는 N+1 문제가 발생합니다. 특히 피드 조회는 빈번하게 호출되므로, 좋아요 여부와 개수를 벌크 쿼리나 캐싱을 통해 한 번에 가져오도록 최적화가 필요합니다.

[gemini-code-assist]

KEY_FORMAT이 :으로 시작하고 RedisPrefix의 접두사들도 :으로 끝나는 경우(예: likePushed:), 생성된 키에 likePushed::memberId와 같이 중복된 구분자가 포함됩니다. 이는 의도치 않은 결과일 수 있으며 키 구조를 불필요하게 복잡하게 만듭니다. 접두사나 포맷 중 한 곳에서 :을 제거하여 일관된 구분자를 사용하도록 개선하는 것이 좋습니다.

[gemini-code-assist]

프로덕션 환경에서는 assert 문이 비활성화될 수 있으므로, connectionFactory가 null이 아님을 보장하기 위해 java.util.Objects.requireNonNull()을 사용하는 것이 더 안전합니다.

Suggested change

	assert connectionFactory != null;
	RedisConnectionFactory connectionFactory = java.util.Objects.requireNonNull(redisTemplate.getConnectionFactory());

[gemini-code-assist]

사용되지 않는 import 문입니다.

[gemini-code-assist]

사용하지 않는 주석 처리된 코드는 제거하여 코드의 가독성을 높이는 것이 좋습니다.