Skip to content

edited_by 매핑과 쓰기를 걷어내고 created_by 하나로 둔다 (3단계) - #1065

Merged
m-a-king merged 1 commit into
devfrom
refactor/1051-wish-item-ref-stage3
Sep 8, 2026
Merged

edited_by 매핑과 쓰기를 걷어내고 created_by 하나로 둔다 (3단계)#1065
m-a-king merged 1 commit into
devfrom
refactor/1051-wish-item-ref-stage3

Conversation

@m-a-king

@m-a-king m-a-king commented Sep 8, 2026

Copy link
Copy Markdown
Collaborator

Situation

Task

  • 코드에서 edited_by 를 완전히 걷어내되 컬럼은 남겨, 1·2·3 단계를 한 번에 promote 해도 배포 창의 영향이 1+2 와 같게 한다.

Action

  • ItemSnapshot 에서 editedBy 생성자 인자·프로퍼티·매핑을 제거했다. 수기 팩토리는 createdBy 만 적는다.
  • 컬럼 edited_by 는 DB 에 남는다. 안 읽는 컬럼이 남는 비용은 없고, DROP 은 옛 코드가 사라진 뒤 아무 릴리즈에나 한 줄 마이그레이션으로 붙인다(엔티티 주석에 명시).
  • 테스트 픽스처의 editedBy 인자와 단언을 createdBy 로 옮겼다.
릴리즈 구성 배포 창에서 옛 컨테이너에 깨지는 것
1+2+3 (이 PR, DROP 없음) 위시·토너먼트 링크 등록 INSERT 만 (item_id NOT NULL)
1+2+3 (DROP 포함이었다면) 스냅샷을 읽는 모든 API

Result

  • promote 는 한 번이면 된다. 트래픽 낮은 시간에 하면 등록 실패는 전환 창 몇 분에 그친다.
  • 남는 것: edited_by DROP 한 줄(나중 아무 릴리즈), 상품별 버전 전체 조회 상한(보류).

연관 이슈

- 2단계까지 아무도 읽지 않게 된 edited_by 를 엔티티에서 떼고 수기 팩토리가 created_by 만 적게 했다. 컬럼은 DB 에 남긴다 - DROP 을 같이 실으면 옛 코드가 스냅샷을 읽는 모든 API 가 배포 창에서 깨져 promote 를 한 번 더 나눠야 하고, 안 읽는 컬럼이 남는 비용은 없다. DROP 은 옛 코드가 사라진 뒤 아무 릴리즈에나 한 줄로 붙인다
- 이로써 1·2·3 단계를 한 릴리즈로 promote 할 수 있고, 배포 창의 영향은 1+2 와 같다(등록 INSERT 만)
- 테스트 픽스처의 editedBy 인자·단언을 createdBy 로 옮겼다
@m-a-king m-a-king added the refactor 구조 개선, 외부 동작 불변 label Sep 8, 2026
@m-a-king m-a-king self-assigned this Sep 8, 2026
@coderabbitai

coderabbitai Bot commented Sep 8, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Advanced

Run ID: 1ab0a126-6044-41e7-8765-958bf4ac736f


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

github-actions Bot commented Sep 8, 2026

Copy link
Copy Markdown

Discord 스레드 연동용 메타데이터입니다. discord-pr-bot 워크플로가 자동 생성하며, 수정·삭제하면 PR 과 Discord 알림 연동이 끊깁니다.

@m-a-king
m-a-king merged commit ce09c35 into dev Sep 8, 2026
7 checks passed
@m-a-king
m-a-king deleted the refactor/1051-wish-item-ref-stage3 branch September 8, 2026 14:27
m-a-king added a commit that referenced this pull request Sep 9, 2026
- created_by 에 흡수(#1054)·매핑 제거(#1065)가 prod 에 promote 된 뒤라 옛 컨테이너가 이 컬럼을 언급하지 않아 배포 창에서도 안전하다. add → backfill → drop 의 마지막 단계
m-a-king added a commit that referenced this pull request Sep 9, 2026
- created_by 에 흡수(#1054)·매핑 제거(#1065)가 prod 에 promote 된 뒤라 옛 컨테이너가 이 컬럼을 언급하지 않아 배포 창에서도 안전하다. add → backfill → drop 의 마지막 단계
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactor 구조 개선, 외부 동작 불변

Projects

None yet

Development

Successfully merging this pull request may close these issues.

위시 참조를 아이템으로 바꾸고 화면값을 행에서 계산한다 (스냅샷 created_by 통합)

1 participant