Skip to content

(재등록) [Refactor] Seeding 속도 개선 - p-limit 병렬 처리 적용 #127

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
2anizirong merged 2 commits into from
Dec 14, 2025
Merged

(재등록) [Refactor] Seeding 속도 개선 - p-limit 병렬 처리 적용 #127

2anizirong merged 2 commits into from
Dec 14, 2025

Conversation

@jack0928
Copy link
Member

@jack0928 jack0928 commented Dec 14, 2025

#️⃣ Related Issues

연관된 모든 Issue를 작성해주세요.
closes 나 resolves prefix를 붙여서 자동으로 Issue가 Close 될 수 있도록 해주세요.

🧑‍💻 작업 내용 및 결과

어떤 작업을 진행했는지 자세하게 작성해주세요.
사진을 첨부하셔도 좋습니다.

  • p-limit(3.1.0) 도입 → 동시 10개 제한 병렬 처리 적용
  • seedSeoulRun4050, seedSeoul50Plus를 Promise.all + p-limit 구조로 리팩터링
  • seeding 실제 실행 결과: 약 30분 내 완료(기존 대비 대폭 단축)
  • Prisma generated 파일의 postinstall: true가 자동 변경되어 false로 수동 복구
  • 기능/로직 변동 없음(seeding 데이터 스키마 동일)

💬 리뷰 시 요청사항 (선택)

Reviewer는 코드 리뷰의 코멘트에 코멘트를 강조하고 싶은 정도를 Pn 규칙에
맞춰서 표기해 주세요.

📝 Checklist

  • Reviewer를 추가했나요?
  • Convention을 준수했나요?

@jack0928
♻️ refactor: 이벤트 seeding 병렬 처리(p-limit) 적용
2a63237 
- p-limit(3.1.0) 적용으로 최대 10개 동시 요청 병렬 처리(CJS 사용 위해 버전 낮춤)
- seedSeoulRun4050, seedSeoul50Plus 함수에 Promise.all + 제한 병렬 구조 도입
- 전체 시딩 소요 시간 약 30분으로 단축 (테스트 완료)
- 주의: pnpm 설치 중 postinstall 옵션이 자동으로 true로 변경됨
@jack0928
♻️ Refactor: Prisma generated 파일의 postinstall 옵션 복구
b4f6e47 
- pnpm 설치 시 자동으로 postinstall 옵션이 true로 변경된 부분을 false로 수동 복구
- Docker 및 CI 환경에서 Prisma generate 중복 실행을 방지하기 위함
@jack0928 jack0928 requested a review from kyeoungwoon December 14, 2025 02:18
@jack0928 jack0928 self-assigned this Dec 14, 2025
@jack0928 jack0928 added D-2 2일 내로 리뷰를 완료해야 합니다. 🧹 Refactor 코드 구조 개선 등 사용자 경험은 동일하지만, 내부 코드 구조 또는 품질을 개선하는 작업입니다. labels Dec 14, 2025
2anizirong
2anizirong approved these changes Dec 14, 2025
View reviewed changes
Copy link
Member

@2anizirong 2anizirong left a comment

Choose a reason for hiding this comment

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

P5: LGTM👍🏻

@2anizirong 2anizirong merged commit d9fd4b6 into develop Dec 14, 2025
2 checks passed
@2anizirong 2anizirong deleted the refactor/#105-optimize-seeding-performance branch December 14, 2025 02:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@2anizirong 2anizirong 2anizirong approved these changes

@kyeoungwoon kyeoungwoon Awaiting requested review from kyeoungwoon kyeoungwoon is a code owner

Assignees

@jack0928 jack0928

Labels

D-2 2일 내로 리뷰를 완료해야 합니다. 🧹 Refactor 코드 구조 개선 등 사용자 경험은 동일하지만, 내부 코드 구조 또는 품질을 개선하는 작업입니다.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Feat] Open API 이벤트 Seeding 기능 추가

3 participants

@jack0928 @2anizirong