Skip to content

Conversation

@wangxuqi
Copy link
Contributor

@wangxuqi wangxuqi commented Jul 5, 2023

Hi, there
This pull request contains two commit:
1. Implement delete interface with optional ids parameter on AnalyticDB.
2. Allow customization of database connection behavior by exposing engine_args parameter in interfaces.
- This commit adds the engine_args parameter to the interfaces, allowing users to customize the behavior of the database connection. The engine_args parameter accepts a dictionary of additional arguments that will be passed to the create_engine function. Users can now modify various aspects of the database connection, such as connection pool size and recycle time. This enhancement provides more flexibility and control to users when interacting with the database through the exposed interfaces.

This commit is related to VectorStores @rlancemartin @eyurtsev

Thank you for your attention and consideration.

wangxuqi added 2 commits July 5, 2023 11:35
…osing engine_args parameter in interfaces. The `engine_args` parameter accepts a dictionary of additional arguments that will be passed to the create_engine function. Users can now modify various aspects of the database connection, such as connection pool size and recycle time. This enhancement provides more flexibility and control to users when interacting with the database through the exposed interfaces.
@vercel
Copy link

vercel bot commented Jul 5, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
langchain ⬜️ Ignored (Inspect) Jul 5, 2023 11:58am

@rlancemartin
Copy link
Contributor

Thanks for adding. Please run "make format" to resolve lint errors.

@wangxuqi
Copy link
Contributor Author

wangxuqi commented Jul 5, 2023

Thanks for adding. Please run "make format" to resolve lint errors.

Sorry for the last commit not meet the code format. Fixed.

Copy link
Contributor

@rlancemartin rlancemartin left a comment

Choose a reason for hiding this comment

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

lgtm

@rlancemartin rlancemartin merged commit cab7d86 into langchain-ai:master Jul 5, 2023
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