Skip to content

feat: 데이터셋 이름에 데이터베이스 정보를 추가 #128

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

ehddnr301
Copy link
Collaborator

#️⃣ Issue Number

📝 요약(Summary)

  • DatahubMetadataFetcher 클래스에서 데이터셋의 이름에 dbt_unique_id를 포함하여 반환하도록 수정
  • 데이터베이스 정보와 테이블 정보를 결합하여 반환 형식 개선

💬 To Reviewers (선택)

image

  • 시각화가 잘 되는지 체크 부탁드립니다.

PR Checklist

  • TBD

reference) How to Code Review

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

- DatahubMetadataFetcher 클래스에서 데이터셋의 이름에 dbt_unique_id를 포함하여 반환하도록 수정
- 데이터베이스 정보와 테이블 정보를 결합하여 반환 형식 개선
@ehddnr301 ehddnr301 linked an issue Jul 6, 2025 that may be closed by this pull request
Copy link
Contributor

@ParkGyeongTae ParkGyeongTae left a comment

Choose a reason for hiding this comment

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

👍 LGTM

@ehddnr301 ehddnr301 merged commit 24b1c3d into master Jul 7, 2025
2 checks passed
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.

Database명이 누락되어 있어서 시각화 오류
2 participants