Skip to content

feat: add MSSQLEngine and MSSQLLoader synced from the mysql code #7

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
merged 8 commits into from
Feb 1, 2024

Conversation

totoleon
Copy link
Contributor

@totoleon totoleon commented Feb 1, 2024

Sync code from langchain-google-cloud-sql-mysql-python with the following tuning

  • changed driver to be pytds
  • updated package names and file names
  • changed from_instance to take username and password, because MSSQL doesn't support IAM auth for now
  • removed the IAM auth and credential provider related code
  • updated integration test to use MSSQL queries
  • updated test assertion to match MSSQL data type
  • updated the cloud build configuration

@totoleon totoleon requested a review from a team as a code owner February 1, 2024 06:28
Copy link
Contributor

@jackwotherspoon jackwotherspoon left a comment

Choose a reason for hiding this comment

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

Overall LGTM, added one open question about argument naming

@totoleon totoleon merged commit 6104306 into staging Feb 1, 2024
@totoleon totoleon deleted the engine branch February 1, 2024 17:40
totoleon added a commit that referenced this pull request Feb 1, 2024
…#8)

* feat: add MSSQLEngine and MSSQLLoader synced from the mysql code (#7)

Sync code from langchain-google-cloud-sql-mysql-python with the following tuning

    changed driver to be pytds
    updated package names and file names
    changed from_instance to take username and password, because MSSQL doesn't support IAM auth for now
    removed the IAM auth and credential provider related code
    updated integration test to use MSSQL queries
    updated test assertion to match MSSQL data type
    updated the cloud build configuration
    updated Cloud Build configure to use secretEnv
---------

Co-authored-by: Hailong Li <hailongli@google.com>
@release-please release-please bot mentioned this pull request Feb 1, 2024
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
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy