refactor: 영상 처리 방식을 메모리에서 디스크로 전환 #48
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request refactors the video answer submission process to use file paths instead of byte arrays, which improves memory efficiency during file handling. The FileUploader has been simplified by removing unused methods, and VideoAnswerAnalyzer now includes more detailed logging for its asynchronous tasks. Feedback focuses on addressing a potential resource leak where temporary files are not deleted after processing, refining exception handling to be more specific, and ensuring consistent S3 key generation by using the UTC timezone.
| Path videoPath = Files.createTempFile("video-", getExtension(video.getOriginalFilename())); | ||
| Files.write(videoPath, videoBytes); | ||
| log.info("[VIDEO] 영상 답변 처리 시작 - questionId: {}, memberId: {}", questionId, memberId); | ||
| Path videoPath = createTempFile(video); |
There was a problem hiding this comment.
생성된 임시 파일(videoPath)이 삭제되지 않고 디스크에 남게 됩니다. 영상 파일은 용량이 크므로 누적될 경우 디스크 풀(Disk Full) 장애를 유발할 수 있습니다. 비동기 작업들이 모두 완료된 후 파일을 삭제하는 로직을 추가해야 합니다.
권장 방법:
VideoAnswerAnalyzer의 메서드들이CompletableFuture<Void>를 반환하도록 변경CompletableFuture.allOf(...).thenRun(() -> { ... 파일 삭제 ... })를 사용하여 모든 작업 완료 후 리소스를 정리
| import io.wisoft.prepair.prepair_api.global.exception.ErrorCode; | ||
| import io.wisoft.prepair.prepair_api.repository.QuestionRepository; | ||
| import java.io.IOException; | ||
|
|
There was a problem hiding this comment.
구체적인 예외 처리를 위해 IOException 임포트가 필요합니다.
| import java.io.IOException; |
| Path videoPath = Files.createTempFile("video-", getExtension(video.getOriginalFilename())); | ||
| video.transferTo(videoPath); | ||
| return videoPath; | ||
| } catch (Exception e) { |
There was a problem hiding this comment.
Exception 대신 구체적인 예외(IOException, IllegalStateException)를 캐치하는 것이 좋습니다. 이는 예기치 않은 런타임 예외를 방지하고 문제 발생 시 원인을 더 명확하게 파악할 수 있게 해줍니다.
| } catch (Exception e) { | |
| } catch (IOException | IllegalStateException e) { |
| String extension = getExtension(originalFilename); | ||
| public String upload(Path videoPath, String contentType, String email) { | ||
| String extension = getExtension(videoPath.getFileName().toString()); | ||
| String key = "interview-video/" + email + "/" + LocalDate.now() + "/" + UUID.randomUUID() + extension; |
There was a problem hiding this comment.
LocalDate.now()는 서버의 시스템 시간대에 의존합니다. 서비스의 일관성을 위해 UTC 시간대를 명시적으로 사용하는 것을 권장합니다.
| String key = "interview-video/" + email + "/" + LocalDate.now() + "/" + UUID.randomUUID() + extension; | |
| String key = "interview-video/" + email + "/" + java.time.LocalDate.now(java.time.ZoneOffset.UTC) + "/" + UUID.randomUUID() + extension; |
1 participant
📌 관련 이슈 (Related Issue)
📝 작업 내용 (Description)
byte[]제거,createTempFile()헬퍼 메서드로 추출VideoAnswerAnalyzer.uploadToS3():byte[]→ Path 파라미터로 변경FileUploader.upload():byte[]→ Path 기반으로 변경,RequestBody.fromFile()사용upload(MultipartFile)오버로드 제거[VIDEO],[VIDEO-S3],[VIDEO-STT],[VIDEO-ANALYSIS]로 통일🔄 변경 유형 (Type of Change)
✅ 체크리스트 (Checklist)
💬 추가 코멘트 (Additional Comments)