Xuanwo commented on PR #7096: URL: https://github.com/apache/opendal/pull/7096#issuecomment-3776946089
> Thanks for the fast follow-up on my issue! I just had a couple of follow-up questions. As I mentioned in the issue, since `PermissionDenied` can happen in contexts other than the file system, I was wondering if we should include those in this fix. This PR fixes all services that use `list` to implement recursive lists. > Also, should we be concerned about other error types as well, or is focusing on `PermissionDenied` sufficient for now? I think `PermissionDenied` is sufficient for now. I assume we will need more complete solutions if we want to make it more generic. For example, we could add a hook here to handle what to do when errors occur: continue, ignore, retry, or raise an error. cc @cjlee38, would you be interested in creating an issue for the more general case if you think it is worthwhile? -- 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]
