Xuanwo
Xuanwo
By the way, I wish the stub generator could convert our Rust comments into Python API comments, so we could generate nice API documentation from them.
> Hey there I'm newer to the data space and am looking to help on a few apache projects. Welcome! > Could I pick up this issue if its not...
> 2\. Call every methods with a retry wrapper. When meeting an unauthorized error, refetch the token and retry the method. Our rest catalog client now has a `authenticate` function....
> 3\. Store `(expired_at, token)` pair in an async `Mutex`, and check whether it's close to expiration when needed. If it's close to expiration, lock the mutex, update the pair,...
The current implementation utilizes `std::sync::Mutex` with a precisely controlled locking scope. https://github.com/apache/iceberg-rust/blob/85627083f0791629be3d73413af936165ebd38eb/crates/catalog/rest/src/client.rs#L88-L179 The worst thing that could happen here is that we might send multiple token refresh requests, which I...
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...
Do we still need this after s3 supports `put-if-not-exists`? `object_store` has removed this support too.
> * It's possible that I'm not using AWS's official S3 service but a third-party adapted S3 service, and this third-party service has this issue? Yep, not all s3 compatible...
Thank you @kingsword09 for working on this, I think we can resolve the conflicts and get ready for merging.
Hi, seems all hdfs tests are failed: https://github.com/apache/opendal/actions/runs/14482506154/job/40622110318?pr=6033