Xuanwo commented on PR #5548: URL: https://github.com/apache/opendal/pull/5548#issuecomment-2606228375
> @Xuanwo Any other questions? There are two things in my mind now: The frist, do we really need an extra `GcsVersionsLister`? It seems nearly the same with `GcsLister`. Then, to support `list_with_deleted`, we may need to use gcs's [`softDeleted`](https://cloud.google.com/storage/docs/json_api/v1/objects/list). However, this API is > If true, only returns [soft-deleted objects](https://cloud.google.com/storage/docs/soft-delete) as part of the objects list response. And > The softDeleted parameter can only be used successfully if the bucket has a soft delete policy. Otherwise, the request fails with a 400 Bad Request error with the reason invalidArgument. If true, versions cannot be set to true. For GCS, `list_with_deleted` is a separate feature that needs to be enabled independently. I believe it would be better to remove this feature for now and reconsider it in the future. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
