Merge branch 'feat/#293' of https://github.com/prgrms-web-devcourse-final-project/WEB7_9_codecrete_BE into feat/#303

Author: ys0221

src/main/java/com/back/web7_9_codecrete_be/domain/artists/repository/ArtistRepository.java

@@ -14,6 +14,15 @@
 @Repository
 public interface ArtistRepository extends JpaRepository<Artist, Long> {
 
+    // 아티스트 상세 조회용 - artistGenres와 genre를 fetch join하여 N+1 문제 방지
+    @Query("""
+        SELECT DISTINCT a FROM Artist a
+        LEFT JOIN FETCH a.artistGenres ag
+        LEFT JOIN FETCH ag.genre
+        WHERE a.id = :id
+    """)
+    java.util.Optional<Artist> findByIdWithArtistGenres(@Param("id") Long id);
+    
     @Query("SELECT a FROM Artist a WHERE a.nameKo IS NULL ORDER BY a.id ASC")
     List<Artist> findByNameKoIsNullOrderByIdAsc(Pageable pageable);
 

src/main/java/com/back/web7_9_codecrete_be/domain/artists/service/ArtistService.java

@@ -100,7 +100,7 @@ public Slice<ArtistListResponse> listArtist(Pageable pageable, User user, Artist
 
     @Transactional(readOnly = true)
     public ArtistDetailResponse getArtistDetail(Long artistId) {
-        Artist artist = artistRepository.findById(artistId)
+        Artist artist = artistRepository.findByIdWithArtistGenres(artistId)
                 .orElseThrow(() -> new BusinessException(ArtistErrorCode.ARTIST_NOT_FOUND));
 
         if (artist.getSpotifyArtistId() == null) {

src/main/java/com/back/web7_9_codecrete_be/domain/artists/service/spotify/application/SpotifyDetailService.java

@@ -49,6 +49,8 @@ public SpotifyArtistDetailCache fetchDetailFromApi(String spotifyArtistId) {
                 throw e;
             } catch (Exception e) {
                 throw new RuntimeException("Exception during getArtist API call", e);
+            } finally {
+                spotifyRateLimiter.release();
             }
         }, "getArtistDetail getArtist spotifyId=" + spotifyArtistId);
 
@@ -80,6 +82,8 @@ private Track[] safeGetTopTracks(SpotifyApi api, String artistId) {
                     throw e;
                 } catch (Exception e) {
                     throw new RuntimeException("Exception during getArtistsTopTracks API call", e);
+                } finally {
+                    spotifyRateLimiter.release();
                 }
             }, "safeGetTopTracks artistId=" + artistId);
         } catch (RuntimeException e) {
@@ -103,6 +107,8 @@ private Paging<AlbumSimplified> safeGetAlbums(SpotifyApi api, String artistId) {
                     throw e;
                 } catch (Exception e) {
                     throw new RuntimeException("Exception during getArtistsAlbums API call", e);
+                } finally {
+                    spotifyRateLimiter.release();
                 }
             }, "safeGetAlbums artistId=" + artistId);
         } catch (RuntimeException e) {

src/main/java/com/back/web7_9_codecrete_be/domain/artists/service/spotify/cache/SpotifyCacheService.java

@@ -41,7 +41,7 @@ public SpotifyArtistDetailCache getCached(String spotifyArtistId) {
 
             return objectMapper.convertValue(cached, SpotifyArtistDetailCache.class);
         } catch (Exception e) {
-            log.warn("Redis 캐시 조회 실패: spotifyArtistId={}", spotifyArtistId, e);
+            log.error("Redis 캐시 조회 실패: spotifyArtistId={}", spotifyArtistId, e);
             return null;
         }
     }

src/main/java/com/back/web7_9_codecrete_be/global/config/RedisConfig.java

@@ -25,7 +25,10 @@ public class RedisConfig {
     @Bean
     public RedisConnectionFactory redisConnectionFactory() {
         final RedisStandaloneConfiguration configuration = new RedisStandaloneConfiguration(host,port);
-        return new LettuceConnectionFactory(configuration);
+        LettuceConnectionFactory factory = new LettuceConnectionFactory(configuration);
+        // Redis 연결 타임아웃 설정 (5초)
+        factory.setTimeout(5000);
+        return factory;
     }
 
     //문자열만 사용할 때(refreshToken)