Conversation
Change-Id: I809cd1d768ad818c6c831bed622d1ab42cd29dbc
|
The failing cli-smoke test to be unrelated to the changes in my PR, and I have opened #154 to fix this. |
|
Codex static security review summary: I think this should also be fix-then-merge. Cloud storage support is useful, but I see a few security / correctness gaps in the current implementation:
Suggested merge criteria:
My conclusion: fix-then-merge. |
|
Thanks @AliceLJY for reviewing. I'm currently working on addressing your comments. May I ask why this PR was closed? I'd be happy to reopen it if needed. |
closes: #75
Add cloud storage support for LanceDB
Changes
Usage
Users can now configure cloud storage connections by providing storage options in the plugin configuration:
This enables the plugin to work with LanceDB Cloud, S3-backed storage, and other remote storage backends without requiring local filesystem access.