Skip to content

Conversation

@jihukimme
Copy link
Member

📝 작업 내용

이번 PR에서는 워크플로우 재개(Resume) 기능의 안정성을 높이기 위한 로직 개선과 프로젝트 문서화를 진행했습니다.

  1. 워크플로우 재개(Resume) 로직 개선

    • 작업 조회 기준 변경 (traceIdworkflowRunId):
      • 기존에는 traceId로 성공한 Job을 조회했으나, 재시도나 중복 실행 시 모호함이 발생할 수 있어 고유한 workflowRunId를 기준으로 조회하도록 변경했습니다.
      • findSuccessfulJobByTraceId를 findSuccessfulJobByWorkflowRunId로 교체했습니다.
    • Task Output 조회 범위 수정:
      • 이전 작업의 결과를 가져오는 로직에서도 workflowRunId 스코프를 적용하여, 해당 실행 회차(Run)의 데이터만 정확히 참조하도록 쿼리 메서드를 개선했습니다.
  2. 코드 리팩토링

    • Workflow 서비스 및 MyBatis 매퍼 전반에 걸쳐 일관된 코드 포맷팅을 적용했습니다.
  3. 문서화 (README.md)

    • 프로젝트 설정, 실행(Getting Started), 배포(Deployment) 가이드를 상세히 추가했습니다.
    • 환경 변수 관리 전략(FastAPI/Spring Boot) 문서를 보완했습니다.

🔗 관련 이슈

  • Closes #이슈번호
  • Related to #이슈번호

💬 추가 요청사항


✅ 체크리스트

코드 품질

  • 커밋 컨벤션 준수 (feat/fix/docs/refactor 등)
  • 불필요한 코드/주석 제거

테스트

  • 로컬 환경에서 동작 확인 완료
  • 기존 기능에 영향 없음 확인

배포 준비

  • 환경변수 추가/변경사항 문서화
  • DB 마이그레이션 필요 여부 확인
  • 배포 시 주의사항 없음

@jihukimme jihukimme self-assigned this Jan 22, 2026
@jihukimme jihukimme added the tech-debt 미뤄둔 기술부채 label Jan 22, 2026
@jihukimme jihukimme merged commit b6af254 into develop Jan 22, 2026
7 checks passed
@jihukimme jihukimme deleted the feature/workflow-resume-recovery branch January 22, 2026 14:15
jihukimme added a commit that referenced this pull request Jan 22, 2026
* README.md 작성 및 DTO 네이밍 컨벤션에 맞게 리팩토링 (#239)

* docs: README.md 작성

* refactor: DTO 네이밍 컨벤션에 따라 리팩토링

* refactor: Code Formatting

* 다이어그램 파일명 수정 (#240)

* docs: README.md 작성

* refactor: DTO 네이밍 컨벤션에 따라 리팩토링

* refactor: Code Formatting

* fix: 다이어그램 파일명 수정

* Spring Quartz에 Cluster를 이용한 schedule, workflow 실시간 반영 (#238)

* feat: applcation-develop.yml에 quartz설정 추가

* feat: QuartzConfig

* feat: Quartz에 스케줄 동기화 추가

* feat: ScheduleController

* feat: Schedule 단건조회, 스케줄 활성상태 변경 추가

* feat: ScheduleService

* feat: ScheduleUpdateDto

* feat: workflowController delete,patch,활성화변경, 스케줄 삭제 추가

* feat: workflowMapper 워크플로우 활성화상태 변경 추가

* feat: Schedule Quartz 실시간 반영 관련 내용 추가

* feat: ScheduleManagementE2eTest 작성중

* chore: spotlessApply

* feat: schedule workflow_id unique 조건 제거

* fix: schedule 관련 파일들 schedule 폴더로 이동

* feat: scheduleE2eTest 수정

* fix: 중복 코드 삭제

* fix: 검증코드 workflowservice -> scheduleService로 이동

* fix: 오타 수정

* fix: 정적 메서드로 변경

* fix: 정적 메서드로 변경

* fix: 인증 중복 코드 제거

* fix: hasSchedules collection으로 변경

* chore: import 추가

* chore: spotlessApply

* refactor: DTO 네이밍 변경

---------

Co-authored-by: jihukimme <jihu0210@naver.com>

* docs: 다이어그램 및 시연 영상 업로드

* docs: 시연 영상 유튜브 링크 업로드

* docs: 시연영상 목차 추가

* 목차 및 각 콘텐츠 앵커 링크 도입 (#242)

* docs: ERD 추가 및 시퀀스 다이어그램 경로 수정 (#244)

* 서버 장애 발생으로 인해 중단된 워크플로우 자동 복구 및 이어하기(Resume) 기능 구현 (#246)

* Release v0.1.0-stable (#245)

* README.md 작성 및 DTO 네이밍 컨벤션에 맞게 리팩토링 (#239)

* docs: README.md 작성

* refactor: DTO 네이밍 컨벤션에 따라 리팩토링

* refactor: Code Formatting

* 다이어그램 파일명 수정 (#240)

* docs: README.md 작성

* refactor: DTO 네이밍 컨벤션에 따라 리팩토링

* refactor: Code Formatting

* fix: 다이어그램 파일명 수정

* Spring Quartz에 Cluster를 이용한 schedule, workflow 실시간 반영 (#238)

* feat: applcation-develop.yml에 quartz설정 추가

* feat: QuartzConfig

* feat: Quartz에 스케줄 동기화 추가

* feat: ScheduleController

* feat: Schedule 단건조회, 스케줄 활성상태 변경 추가

* feat: ScheduleService

* feat: ScheduleUpdateDto

* feat: workflowController delete,patch,활성화변경, 스케줄 삭제 추가

* feat: workflowMapper 워크플로우 활성화상태 변경 추가

* feat: Schedule Quartz 실시간 반영 관련 내용 추가

* feat: ScheduleManagementE2eTest 작성중

* chore: spotlessApply

* feat: schedule workflow_id unique 조건 제거

* fix: schedule 관련 파일들 schedule 폴더로 이동

* feat: scheduleE2eTest 수정

* fix: 중복 코드 삭제

* fix: 검증코드 workflowservice -> scheduleService로 이동

* fix: 오타 수정

* fix: 정적 메서드로 변경

* fix: 정적 메서드로 변경

* fix: 인증 중복 코드 제거

* fix: hasSchedules collection으로 변경

* chore: import 추가

* chore: spotlessApply

* refactor: DTO 네이밍 변경

---------

Co-authored-by: jihukimme <jihu0210@naver.com>

* docs: 다이어그램 및 시연 영상 업로드

* docs: 시연 영상 유튜브 링크 업로드

* docs: 시연영상 목차 추가

* 목차 및 각 콘텐츠 앵커 링크 도입 (#242)

* docs: ERD 추가 및 시퀀스 다이어그램 경로 수정 (#244)

---------

Co-authored-by: bwnfo3 <142577603+bwnfo3@users.noreply.github.com>
Co-authored-by: Yousung Jung <jys01012@gmail.com>

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* README.md 업데이트

* feat: 워크플로우 비정상 종료 복구 프로세스 추가 및 Mapper 메서드 확장

* feat: 성공한 Job 확인용 Mapper 메서드 추가 및 워크플로우 실행 로직 수정

* refactor: Code formatting and alignment adjustments across core classes

* refactor: Code formatting and alignment adjustments across core classes

---------

Co-authored-by: bwnfo3 <142577603+bwnfo3@users.noreply.github.com>
Co-authored-by: Yousung Jung <jys01012@gmail.com>

* refactor: 워크플로우 재개(Resume) 로직 개선 및 README 문서 보완 (#248)

* refactor: replace findSuccessfulJobByTraceId with findSuccessfulJobByWorkflowRunId

* refactor: update task output retrieval to use workflowRunId scope and refine query methods

* docs: update README with environment variables, setup, and deployment steps

* refactor: improve code formatting across Workflow services and mappers

---------

Co-authored-by: bwnfo3 <142577603+bwnfo3@users.noreply.github.com>
Co-authored-by: Yousung Jung <jys01012@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tech-debt 미뤄둔 기술부채

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants