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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Feature/80 add connectors and metadata #104

wants to merge 4 commits into 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
@YuujinHwang
Add additional dependencies to pyproject.toml
19a6cbf 
- Added specific version of transformers (==4.51.2)
- Added clickhouse-driver (>=0.2.9,<0.3.0)
- Added numpy (<2.0)
add database utilities
e302776
add database utilities
f9a9453
change connector instance in lang2sql.py
37b9c5e
@ehddnr301
Copy link
Collaborator

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

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@YuujinHwang @ehddnr301