fix: DataSourceConfig 클래스 이름을 DataSourceProxyConfig로 revert 하여 충돌 유도 #617
+1
−1
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.
관련 이슈
작업 내용
반복되는 문제 발생 죄송합니다...
현재 release 브랜치에 존재하는 DataSourceProxyConfig 클래스와
develop에 존재하는 DataSourceConfig 클래스(DataSourceProxyConfig 를 수정한 클래스)
가 relesae 로 merge 시 공존해버리는 문제가 발생하여 빈 등록 문제가 생겼습니다.
이를 해결하기 위해, develop에서 가지 친 브랜치에 relesae 브랜치를 rebase 해서 공존하는 DataSource 클래스 중 이전 버전의 클래스를 변경하도록 이력을 남기려고 했으나, develop 브랜치에서 release 브랜치의 커밋내역을 알게 되는 문제가 발생하여 다른 방식으로 처리 해야할 듯 합니다.
그래서 현재 문제는 클래스 이름이 달라서 충돌이 나지 않고, 각 datasource 들이 공동으로 존재하는 문제이므로, 충돌을 발생시키기 위해 다시 클래스 이름을 동일하게 하여 develop -> release 로 merge 시 충돌을 발생시켜 conflict 를 해결하는 방식이 가능할 듯 하여 이름을 revert 하였습니다.
특이 사항
리뷰 요구사항 (선택)