apache / apache/iceberg-python

refactor sigv4 out of rest catalog

Open
#3,410 1 comment 1 reaction 0 assignees View on GitHub
Dominant language
Python
Stars
1.1k
Forks
581
Avg merge
1d 17h
Merged PRs (30d)
78

Description

https://github.com/apache/iceberg-python/blob/8055a60af5d32ec4d2e1d02cd020a78273785558/pyiceberg/catalog/rest/__init__.py#L764-L820

into new file `pyiceberg/catalog/rest/sigv4.py`

and generally improve sigv4 implementation
- align on retry behavior (#3008)
- fix request signing (#3120)

Contributor guide

No contributing guide indexed for this repository

Research direction

Start with the SigV4 implementation in pyiceberg/catalog/rest/__init__.py at lines 764-820 and compare the requested destination, pyiceberg/catalog/rest/sigv4.py. Read issues #3008 and #3120 for the retry and request-signing requirements. Done means the implementation is separated and those two referenced concerns are addressed.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
backend-api-design, security
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.