langchain
langchain copied to clipboard
fix SQL toolkit table listing tool name
- Description: This PR fixes the erroneous naming convention of the
list_tables_sql_dbtool used for querying the list of tables present in a database. This is a minor fix which renames all the occurrences of the above tool with the one described in the tool's description. - Issue: Fixes #7440
- Dependencies: None
- Maintainer: @hinthornw
The latest updates on your projects. Learn more about Vercel for Git ↗︎
| Name | Status | Preview | Comments | Updated (UTC) |
|---|---|---|---|---|
| langchain | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Jul 9, 2023 6:16pm |
@hinthornw seems spark sql tool already has this name, should we rename the spark one to indicate it's spark?
@baskaryan Should there be 2 separate tools for the same job?
@baskaryan FYI ^
Closing because the PR wouldn't line up with the current directory structure of the library (would need to be in /libs/langchain/langchain instead of /langchain). Feel free to reopen against the current head if it's still relevant!