Skip to content

Conversation

@alamb
Copy link
Contributor

@alamb alamb commented Dec 18, 2025

Which issue does this PR close?

Rationale for this change

Update readme to reflect current plans

What changes are included in this PR?

Are there any user-facing changes?

@alamb alamb merged commit 24e5fbc into apache:main Dec 19, 2025
8 checks passed
@alamb
Copy link
Contributor Author

alamb commented Dec 19, 2025

Thank you @kylebarron

@alamb alamb deleted the alamb/update_release_schemdule branch December 19, 2025 13:54
/// |
/// ```
///
/// 3. Convert `ObjectStore::delete` to [`ObjectStore::delete_stream`] (see documentation
Copy link
Contributor

Choose a reason for hiding this comment

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

Why do we direct users to delete_stream and copy_opts and not ObjectStoreExt?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This was meant for people who have implemented ObjectStore (not people who are calling)

What has happened is they now need to implement ObjectStore::delete_stream

The documentation on ObjectStore::delete_stream tries to explain what happened (and tries to explain how to port the code)

/// Note: Before version 0.13, `delete_stream` has a default implementation
/// that deletes each object with up to 10 concurrent requests. This default
/// behavior has been removed, and each implementation must now provide its
/// own `delete_stream` implementation explicitly. The following example
/// shows how to implement `delete_stream` to get the previous default
/// behavior.

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.

3 participants