Skip to content

Commit 1e4e92c

Browse files
committed
REST: Remove version HTTP header
This was added way ago, but I don't think it makes any sense for two reasons: - It was never updated properly :) - The spec is not versioned with the Java release Better to clean this up
1 parent 4b961f7 commit 1e4e92c

File tree

2 files changed

+0
-4
lines changed

2 files changed

+0
-4
lines changed

pyiceberg/catalog/rest/__init__.py

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -80,8 +80,6 @@
8080
if TYPE_CHECKING:
8181
import pyarrow as pa
8282

83-
ICEBERG_REST_SPEC_VERSION = "0.14.1"
84-
8583

8684
class Endpoints:
8785
get_config: str = "config"
@@ -485,7 +483,6 @@ def _config_headers(self, session: Session) -> None:
485483
header_properties = get_header_properties(self.properties)
486484
session.headers.update(header_properties)
487485
session.headers["Content-type"] = "application/json"
488-
session.headers["X-Client-Version"] = ICEBERG_REST_SPEC_VERSION
489486
session.headers["User-Agent"] = f"PyIceberg/{__version__}"
490487
session.headers.setdefault("X-Iceberg-Access-Delegation", ACCESS_DELEGATION_DEFAULT)
491488

tests/catalog/test_rest.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,6 @@
5858

5959
TEST_HEADERS = {
6060
"Content-type": "application/json",
61-
"X-Client-Version": "0.14.1",
6261
"User-Agent": f"PyIceberg/{pyiceberg.__version__}",
6362
"Authorization": f"Bearer {TEST_TOKEN}",
6463
"X-Iceberg-Access-Delegation": "vended-credentials",

0 commit comments

Comments
 (0)