Skip to content
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

Pass engine_args to create_engine in get_embedding_column_definition #27

Conversation

michaelwnyc
Copy link
Contributor

Pass engine_args to create_engine in get_embedding_column_definition.

Encountered this error when connecting to tidbcloud serverless cluster:

sqlalchemy.exc.OperationalError: (pymysql.err.OperationalError) (2003, "Can't connect to MySQL server on 'gateway01.eu-central-1.prod.aws.tidbcloud.com' ([SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1000))")

@IANTHEREAL
Copy link
Collaborator

@michaelwnyc thanks for your contribution. Can you fix the lint ci issue?

@michaelwnyc
Copy link
Contributor Author

@IANTHEREAL It should have been fixed

@IANTHEREAL IANTHEREAL merged commit 1eeab27 into pingcap:main Apr 22, 2024
1 of 2 checks passed
@IANTHEREAL
Copy link
Collaborator

@michaelwnyc Thanks again

@michaelwnyc
Copy link
Contributor Author

@IANTHEREAL Any chance to release a new version and also update llama-index-vector-stores-tidbvector's version list?

@IANTHEREAL
Copy link
Collaborator

@michaelwnyc we will release it ASAP in this week

@IANTHEREAL
Copy link
Collaborator

IANTHEREAL commented Apr 23, 2024

@michaelwnyc We had released the update in tidb-vector 0.0.8. You can now update 'tidb-vector' in your environment. Because the 'llama-index-vector-stores-tidbvector' requires 'tidb-vector' to be version '>=0.0.3,<1.0.0', it will also work.

If you have any other usage issues, please feel free to contact me either through an issue or by email at [email protected].

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