Skip to content

Conversation

ParkGyeongTae
Copy link
Contributor

#️⃣ Issue Number

📝 요약(Summary)

  • 기존에 분산되어 있던 data_utils/, db_utils/ 디렉토리를 utils/ 디렉토리로 통합하여 모듈 구조를 개선했습니다.
  • 관련 import 경로를 모두 utils. 네임스페이스로 변경하여 일관성을 확보했습니다.
  • 기능적 변경 없이 코드베이스의 구조적 개선을 통해 유지보수성을 향상시켰습니다.

💬 To Reviewers (선택)

  • import 경로 변경이 올바르게 적용되었는지 확인 부탁드립니다.
  • 기존 기능에 영향을 주지 않았는지 검증 부탁드립니다.
  • 새로운 utils/ 디렉토리 구조가 논리적으로 잘 구성되었는지 확인 부탁드립니다.

PR Checklist

  • 기존 기능 테스트 (DB 연결, 데이터 소스 관리 등)
  • import 경로 변경 검증
  • 설정 UI 동작 확인
  • Docker 관련 설정 검증 (pgvector 추가)

reference) How to Code Review

  • 따봉(👍): 리뷰어가 리뷰이의 코드에서 칭찬의 의견을 남기고 싶을 때 사용합니다.
  • 느낌표(❗): 리뷰어가 리뷰이에게 필수적으로 코드 수정을 요청할 때 사용합니다.
  • 물음표 (❓): 리뷰어가 리뷰이에게 의견을 물어보고 싶을 때 사용합니다.
  • 알약 (💊): 리뷰어가 리뷰이의 코드에서 개선된 방법을 제안하지만 그것의 반영이 필수까지는 아닐 때 사용합니다.

Copy link
Collaborator

@ehddnr301 ehddnr301 left a comment

Choose a reason for hiding this comment

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

👍 전체적으로 좋은 방향성이라고 생각됩니다아!

❗일부 변경이 필요한 지점이 보이는것같은데 확인부탁드립니다아!

finally:
try:
cursor.close()
except e:
Copy link
Collaborator

Choose a reason for hiding this comment

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

❗요부분 Exception 처리가 조금 이상한듯 합니다아!

  • Trino 는 현재 에러는 발생하지 않는데 그냥 단순 문법확인차원 입니다아. cursor.close() 동작시에 에러가 발생해야만 문법에러가 발생할듯 하네요!

try:
module_name = f"utils.databases.connector.{db_type}_connector"
module = importlib.import_module(module_name)
connector_class = getattr(module, f"{db_type.capitalize()}Connector")
Copy link
Collaborator

Choose a reason for hiding this comment

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

❗이 부분이 capitalize() 로 처리하게되면 ClickHouseConnector, Duckdb 같은 케이스를 커버하지 못할것으로 보입니다.

Copy link
Collaborator

@ehddnr301 ehddnr301 left a comment

Choose a reason for hiding this comment

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

👍 고생하셨습니다!!

@ParkGyeongTae ParkGyeongTae merged commit d248a92 into master Oct 6, 2025
1 check passed
@ParkGyeongTae ParkGyeongTae self-assigned this Oct 6, 2025
@ehddnr301 ehddnr301 deleted the 188-refactor-utils-modules branch October 11, 2025 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants