Skip to content

Conversation

@05AM
Copy link
Contributor

@05AM 05AM commented Jun 11, 2025

💡 유형

  • 새로운 기능 추가

💁 해결하려는 문제를 적어주세요

  • 실제로 업로드 되지 않은 파일 메타데이터를 삭제하는 배치를 구현했습니다.
  • 업로드 된 파일의 현황을 파악할 수 있도록 합니다.

🙋 중점적으로 리뷰 했으면 하는 부분이 있다면 적어주세요

  • 혹시 JobRunner의 역할이 애플리케이션 실행마다 하나의 job을 실행하는 것일까요? job이 여러 개로 늘어남에 따라 어떤 job을 주입해야하는지 알 수 없어 에러가 발생하여 주석 처리 해두었습니다.
    image
  • 꼭 필요한 작업이 아니라면 제거하거나, job을 명시하는 것이 좋을 것 같습니다.

@05AM 05AM requested a review from DuFong June 11, 2025 08:32
@05AM 05AM self-assigned this Jun 11, 2025
@05AM 05AM added D-4 4일 전 까지 리뷰해주세요 feat 새로운 기능 labels Jun 11, 2025
@github-actions github-actions bot added D-3 3일 전 까지 리뷰해주세요 D-2 2일 전 까지 리뷰해주세요 D-1 1일 전 까지 리뷰해주세요 D-0 바로 리뷰가 필요해요 and removed D-4 4일 전 까지 리뷰해주세요 D-3 3일 전 까지 리뷰해주세요 D-2 2일 전 까지 리뷰해주세요 D-1 1일 전 까지 리뷰해주세요 labels Jun 11, 2025
@DuFong DuFong deployed to develop June 15, 2025 07:17 — with GitHub Actions Active
Copy link
Collaborator

@DuFong DuFong left a comment

Choose a reason for hiding this comment

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

고생하셨습니다.

public class JobRunner implements ApplicationRunner {

private final JobLauncher jobLauncher;
private final Job job;
Copy link
Collaborator

Choose a reason for hiding this comment

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

신규 Job Bean deleteOldUnuploadedFilesJob이 추가 되면서
인스타배치 Job과 메타데이터 Job이 충돌하였습니다.

  • @Qualifier 를 통해 조치 가능하나, 인스타그램 배치를 위한 Task를 별도 분리하여 조치하였습니다
@Qualifier("instagramMediaJob")
private final Job job;

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

D-0 바로 리뷰가 필요해요 feat 새로운 기능

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants