Skip to content

Conversation

@sukangpunch
Copy link
Contributor

관련 이슈

  • resolves: #이슈 번호

작업 내용

반복되는 문제 발생 죄송합니다...

현재 release 브랜치에 존재하는 DataSourceProxyConfig 클래스와
develop에 존재하는 DataSourceConfig 클래스(DataSourceProxyConfig 를 수정한 클래스)
가 relesae 로 merge 시 공존해버리는 문제가 발생하여 빈 등록 문제가 생겼습니다.

이를 해결하기 위해, develop에서 가지 친 브랜치에 relesae 브랜치를 rebase 해서 공존하는 DataSource 클래스 중 이전 버전의 클래스를 변경하도록 이력을 남기려고 했으나, develop 브랜치에서 release 브랜치의 커밋내역을 알게 되는 문제가 발생하여 다른 방식으로 처리 해야할 듯 합니다.

그래서 현재 문제는 클래스 이름이 달라서 충돌이 나지 않고, 각 datasource 들이 공동으로 존재하는 문제이므로, 충돌을 발생시키기 위해 다시 클래스 이름을 동일하게 하여 develop -> release 로 merge 시 충돌을 발생시켜 conflict 를 해결하는 방식이 가능할 듯 하여 이름을 revert 하였습니다.

특이 사항

리뷰 요구사항 (선택)

@sukangpunch sukangpunch self-assigned this Jan 22, 2026
@sukangpunch sukangpunch added the 버그 Something isn't working label Jan 22, 2026
@coderabbitai
Copy link

coderabbitai bot commented Jan 22, 2026

Walkthrough

데이터 소스 설정 관련 클래스의 이름 정리 작업입니다.

  1. 클래스명 변경: DataSourceConfig 에서 DataSourceProxyConfig 로 이름 변경
  2. 파일명 동기화: 클래스 이름과 일치하도록 파일도 함께 변경
  3. 기능 보존: Lombok 애너테이션과 필드는 그대로 유지되며 동작상 변화 없음

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested reviewers

  • wibaek
  • Hexeong
  • JAEHEE25
  • lsy1307
  • Gyuhyeok99
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목이 변경 사항을 명확하게 설명합니다. DataSourceConfig 클래스를 DataSourceProxyConfig로 되돌리는 작업과 그 의도(충돌 유도)를 구체적으로 나타냅니다.
Description check ✅ Passed PR 설명이 템플릿 구조를 따르고 있으며, 문제 상황, 시도한 해결책, 최종 조치를 상세히 설명합니다. 필수 섹션인 '관련 이슈', '작업 내용', '특이 사항'을 모두 포함하고 있습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

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 and usage tips.

Copy link
Member

@whqtker whqtker left a comment

Choose a reason for hiding this comment

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

확인했습니다 ~

파일명 변경과 새로운 파일 추가를 별도의 커밋으로 분리했으면 괜찮았으려나요 ..?

@sukangpunch sukangpunch merged commit aaafc36 into solid-connection:develop Jan 23, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

버그 Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants