Skip to content

added support for AWS S3#184

Merged
sonam-pankaj95 merged 1 commit into
StarlightSearch:mainfrom
taradepan:AWS-S3
Nov 6, 2025
Merged

added support for AWS S3#184
sonam-pankaj95 merged 1 commit into
StarlightSearch:mainfrom
taradepan:AWS-S3

Conversation

@taradepan

Copy link
Copy Markdown
Contributor

No description provided.

@sonam-pankaj95

Copy link
Copy Markdown
Collaborator

Thanks @taradepan, will get back shortly.

@sonam-pankaj95

Copy link
Copy Markdown
Collaborator

Hey I tested the python example file.

it's throwing this error. S3 Client is external library?

image

@sonam-pankaj95

Copy link
Copy Markdown
Collaborator

I think I need to do maturin build because it's rust internal code.

@taradepan

Copy link
Copy Markdown
Contributor Author

Hey, s3 client in not an external library. you need to use maturin develop

@sonam-pankaj95

sonam-pankaj95 commented Nov 5, 2025

Copy link
Copy Markdown
Collaborator

Update: Yes, the maturin build initiated S3 bucket files fetch. But it's showing access denied for me. I think that might be my account s3 problem, and we should go ahead and merge.
CC: @akshayballal95

image

@sonam-pankaj95 sonam-pankaj95 merged commit 6bc32b2 into StarlightSearch:main Nov 6, 2025
5 checks passed
@sonam-pankaj95

Copy link
Copy Markdown
Collaborator

Thank you, @taradepan, for your contribution; this issue was at the top of our list.

@taradepan

Copy link
Copy Markdown
Contributor Author

Happy to contribute 🙌

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