Xuanwo commented on issue #6076: URL: https://github.com/apache/opendal/issues/6076#issuecomment-2823004060
Hi, @chitralverma, thanks a lot for working on this. > While doing this i came across a few limitation which need Debug to be implemented for Reader and Writer. The requirement is from `polars`? I'm open to implement `Debug` for `Reader` and `Writer` and maybe expose some meaningful internal states. > When reader is created, maybe a stat call can be triggered eagerly which exposes the Metadata object as a field in reader. The reader can fail eagerly if the object doesn't exist on this basis. Maybe we can have a new option for this, but I'm not sure if it's a good idea. > * as i remember, this eager call is anyways triggered when user passes byte range as `..`, so might as well do this beforehand. The `stat` call only occurs when users input both `..` and `chunk/range` simultaneously. This means that if users want to read from the beginning to the end, no `stat` call is triggered. > Metadata has a lot of info already but some things like scheme, prefix, and fully qualified path are still missing, but they're available in the ReaderContext. They can be retrieved from `op.info()`, so having a duplicate in `Metadata` seems a bit redundant to me. -- 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]
