Skip to content

Feature/80 add connectors and metadata #104

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

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

YuujinHwang
Copy link

#️⃣ Issue Number

📝 요약(Summary)

  • 대중적으로 사용하는 데이터베이스들에 대한 커넥터 및 sql execution을 추가했습니다.

💬 To Reviewers (선택)

  • 경태님이 올려주신 llm_response_parser와 일부 기능이 겹칩니다. (Response 에서 ```sql 블럭 추출)
    머지 시 주의 필요합니다.
  • 고봉밥이 될 것 같아서 잘라 올립니다.

PR Checklist

  • clickhouse
  • databricks
  • duckdb
  • mariadb
  • mysql
  • oracle
  • postgresql
  • snowflake
  • sqlite

reference) How to Code Review

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

YuujinHwang and others added 4 commits May 12, 2025 06:48
- Added specific version of transformers (==4.51.2)
- Added clickhouse-driver (>=0.2.9,<0.3.0)
- Added numpy (<2.0)
@ehddnr301
Copy link
Collaborator

👍 DB연결관련해서 더 지원이 확장되어 좋습니다.

❗Black 으로 formatting이 필요합니다.

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.

👍 테스트해봤습니다! 현재 쿼리 생성때 database가 명시가 안돼서 에러는 나지만 clickhouse 연결은 확인했습니다.

@YuujinHwang

  1. 수정사항 확인해주시면 머지해도 될듯합니다! clickhouse 쿼리하는 부분이 에러가 나서 아래와 같이 수정했습니다.
            result = self.client.query_dataframe(sql)
            return result
  1. pyproject.toml 파일의 내용을 requirements.txt 로 넣어두었습니다. (아직 배포관련 사항이 setup.py로 되어있어서요ㅜㅜ)
  2. 그외에는 conflict 해결을 위해 master branch 내용을 merge 했습니다.

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