apache / apache/iceberg-python
feat: refactor rest signer in fileio
- Ngôn ngữ chính
- Python
- Star
- 1.1k
- Fork
- 581
- Merge trung bình
- 1 ngày 17 giờ
- Pull request đã merge (30 ngày)
- 78
Mô tả
### Feature Request / Improvement
Context: https://github.com/apache/iceberg-python/pull/2846#discussion_r2645920570
Currently S3 signer is tightly coupled with FileIO. This is awkward since the S3 Signer requires interaction with remote service (i.e. REST Catalog) and reuse its auth manager.
We can refactor here and create the signer in REST Catalog and pass to FileIO by overriding its `_load_file_io` function.
Also see [_init_sigv4](https://github.com/apache/iceberg-python/blob/a99dcada431a06329959cb7f71bf961343256bb0/pyiceberg/catalog/rest/__init__.py#L409-L459) for inspiration
Hướng dẫn đóng góp
Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này
Hướng nghiên cứu
Đọc phần thảo luận trong PR 2846 và phần triển khai `_init_sigv4` trong `pyiceberg/catalog/rest/__init__.py`. Theo dõi đường đi `_load_file_io` của REST Catalog và cách S3 signer hiện đang liên kết với FileIO. Hoàn thành khi REST Catalog tạo signer bằng auth manager của nó và truyền signer đó cho FileIO thông qua override.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- python
- Lĩnh vực
- api, authentication, backend
- Loại issue
- Tái cấu trúc
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức độ hoạt động
- Ít trao đổi
- Độ rõ ràng
- Khá rõ ràng
- Mức phù hợp với người mới
- 48/100