Skip to content

Documentation for backend support of bulk delete#571

Merged
alamb merged 1 commit into
apache:mainfrom
jayvdb:delete-stream-docs
Dec 18, 2025
Merged

Documentation for backend support of bulk delete#571
alamb merged 1 commit into
apache:mainfrom
jayvdb:delete-stream-docs

Conversation

@jayvdb

@jayvdb jayvdb commented Dec 10, 2025

Copy link
Copy Markdown
Contributor

Which issue does this PR close?

Closes #527

Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?

Comment thread src/parse.rs
/// # Supported formats:
/// - `file:///path/to/my/file` -> [`LocalFileSystem`]
/// - `memory:///` -> [`InMemory`]
/// - `s3://bucket/path` -> [`AmazonS3`](crate::aws::AmazonS3) (also supports `s3a`)

@kylebarron kylebarron Dec 11, 2025

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Why remove all these hyperlinks?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

They are errors when running cargo doc

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

You need to run with --all-features

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

ahh, of course! ok , I'll revert. Thanks!

@jayvdb jayvdb force-pushed the delete-stream-docs branch from 8d39335 to 25ed2ab Compare December 11, 2025 18:37
@jayvdb jayvdb requested a review from kylebarron December 11, 2025 20:23
@alamb alamb merged commit 7b09e4e into apache:main Dec 18, 2025
8 checks passed
@alamb

alamb commented Dec 18, 2025

Copy link
Copy Markdown
Contributor

Thanks @kylebarron and @jayvdb

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.

Add documentation for what backends support bulk delete (ObjectStore::delete_stream)

3 participants