옛 정책 테이블 extraction_platform_policies 제거 - #964
Merged
Merged
Conversation
- 단계 배포(add → 양쪽 공존 → drop)의 마지막 단계다. #955 에서 새 테이블을 만들고 코드 참조를 전부 끊었지만, blue-green 공존 구간의 구버전 인스턴스가 부팅 때 이 테이블을 읽어(@PostConstruct findAll) 그때는 남겨 뒀다. 그 배포가 dev·prod 를 한 바퀴 돌아 이제 지운다 - 행을 옮기지 않는다. 옛 route 3값은 새 축으로 기계적 매핑이 성립하지 않아(같은 HEADLESS_FIRST 라도 kream 은 차단, store.kakao 는 아님) 새 테이블을 비운 채 시작했고, 여기서 되살릴 근거도 없다. reason 에 남아 있던 실측 메모를 보존할지 물었으나 새로 가는 쪽으로 결정했다 - 별도 테스트는 두지 않는다. 통합 테스트가 빈 DB 에 전체 마이그레이션을 새로 적용하므로 이 DROP 의 순서·성립 여부가 그 실행으로 검증된다
|
Discord 스레드 연동용 메타데이터입니다. discord-pr-bot 워크플로가 자동 생성하며, 수정·삭제하면 PR 과 Discord 알림 연동이 끊깁니다. |
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: Path: .coderabbit.yml Review profile: CHILL Plan: Pro Plus Run ID: 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. Comment |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Situation
DomainAccess { ALLOWED, BLOCKED }하나로 바꾸면서 새 테이블을 만들고 코드의 참조를 전부 끊었지만, 옛 테이블extraction_platform_policies는 DB 에 그대로 남겨 뒀다.@PostConstruct의findAll) 죽는다.Task
Action
DROP TABLE IF EXISTS extraction_platform_policies마이그레이션 하나. 컬럼 단위로 지울 필요가 없어 테이블째 떨어뜨린다.route3값은 새 축으로 기계적 매핑이 성립하지 않는다. 같은HEADLESS_FIRST라도 kream 은 차단이고 store.kakao 는 차단이 아니라 갈 곳이 다르다. 플랫폼 정책을 허용·차단 한 축으로 교체 #955 에서 새 테이블을 비운 채 시작한 것과 같은 판단이다.reason컬럼에 "403 봇 차단" 같은 실측 메모가 남아 있어 보존 여부를 저울질했다. 새 판단은 새 근거로 세우기로 하고 함께 버린다.CREATE들보다 뒤에 놓였는지와 실제로 성립하는지가 그 실행으로 검증된다.Result
domain_access_policies하나로 정리된다.연관 이슈