[DRAFT] Make Storage a trait

.asf.yaml

@@ -40,7 +40,7 @@ github:
 
       required_pull_request_reviews:
         required_approving_review_count: 1
-        dismiss_stale_reviews: true
+        dismiss_stale_reviews: false
 
       required_linear_history: true
   del_branch_on_merge: true

.github/workflows/bindings_python_ci.yml

@@ -56,17 +56,21 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v5
+      - uses: astral-sh/setup-uv@v6
+        with:
+          version: "0.9.3"
+          enable-cache: true
       - name: Install tools
         run: |
-          pip install ruff
+          uv tool install ruff
       - name: Check format
         working-directory: "bindings/python"
         run: |
-          ruff format . --diff
+          uvx ruff format . --diff
       - name: Check style
         working-directory: "bindings/python"
         run: |
-          ruff check .
+          uvx ruff check .
 
   test:
     runs-on: ${{ matrix.os }}
@@ -80,17 +84,28 @@ jobs:
       - uses: actions/checkout@v5
       - uses: actions/setup-python@v6
         with:
-          python-version: 3.9
+          python-version: 3.12
       - uses: PyO3/maturin-action@v1
         with:
           working-directory: "bindings/python"
           command: build
           args: --out dist --sdist
+      - uses: astral-sh/setup-uv@v6
+        with:
+          version: "0.9.3"
+          enable-cache: true
+      - name: Sync dependencies
+        working-directory: "bindings/python"
+        shell: bash
+        run: |
+          make install
+      - name: Install built wheel
+        working-directory: "bindings/python"
+        shell: bash
+        run: |
+          uv pip install --reinstall dist/pyiceberg_core-*.whl
       - name: Run tests
         working-directory: "bindings/python"
         shell: bash
         run: |
-          set -e
-          pip install hatch==1.12.0
-          hatch run dev:pip install dist/pyiceberg_core-*.whl --force-reinstall
-          hatch run dev:test
+          make test

.github/workflows/ci.yml

@@ -58,7 +58,7 @@ jobs:
         uses: ./.github/actions/setup-builder
 
       - name: Check License Header
-        uses: apache/skywalking-eyes/header@v0.7.0
+        uses: apache/skywalking-eyes/header@v0.8.0
 
       - name: Check toml format
         run: make check-toml
@@ -89,6 +89,18 @@ jobs:
           - macos-latest
           - windows-latest
     steps:
+      - name: Maximize build space (Ubuntu)
+        if: matrix.os == 'ubuntu-latest'
+        uses: easimon/maximize-build-space@master
+        with:
+          remove-dotnet: "true"
+          remove-android: "true"
+          remove-haskell: "true"
+          remove-codeql: "true"
+          remove-docker-images: "true"
+          root-reserve-mb: 10240
+          temp-reserve-mb: 10240
+
       - uses: actions/checkout@v5
 
       - name: Setup Rust toolchain

.github/workflows/release_python.yml

@@ -107,7 +107,7 @@ jobs:
           command: sdist
           args: -o dist
       - name: Upload sdist
-        uses: actions/upload-artifact@v4
+        uses: actions/upload-artifact@v5
         with:
           name: wheels-sdist
           path: bindings/python/dist
@@ -159,7 +159,7 @@ jobs:
           command: build
           args: --release -o dist
       - name: Upload wheels
-        uses: actions/upload-artifact@v4
+        uses: actions/upload-artifact@v5
         with:
           name: wheels-${{ matrix.os }}-${{ matrix.target }}
           path: bindings/python/dist
@@ -178,7 +178,7 @@ jobs:
 
     steps:
       - name: Download all the dists
-        uses: actions/download-artifact@v5
+        uses: actions/download-artifact@v6
         with:
           pattern: wheels-*
           merge-multiple: true

.github/workflows/release_python_nightly.yml

@@ -56,7 +56,7 @@ jobs:
           args: -o dist
 
       - name: Upload sdist
-        uses: actions/upload-artifact@v4
+        uses: actions/upload-artifact@v5
         with:
           name: wheels-sdist
           path: bindings/python/dist
@@ -102,7 +102,7 @@ jobs:
           args: --release -o dist
 
       - name: Upload wheels
-        uses: actions/upload-artifact@v4
+        uses: actions/upload-artifact@v5
         with:
           name: wheels-${{ matrix.os }}-${{ matrix.target }}
           path: bindings/python/dist
@@ -120,7 +120,7 @@ jobs:
 
     steps:
       - name: Download all the dists
-        uses: actions/download-artifact@v5
+        uses: actions/download-artifact@v6
         with:
           pattern: wheels-*
           merge-multiple: true

.licenserc.yaml

@@ -21,16 +21,17 @@ header:
     copyright-owner: Apache Software Foundation
 
   paths-ignore:
-    - 'LICENSE'
-    - 'NOTICE'
-    - '.gitattributes'
-    - '**/*.json'
+    - "LICENSE"
+    - "NOTICE"
+    - ".gitattributes"
+    - "**/*.json"
     # Generated content by mdbook
-    - 'website/book'
+    - "website/book"
     # Generated content by scripts
-    - '**/DEPENDENCIES.*.tsv'
+    - "**/DEPENDENCIES.*.tsv"
     # Release distributions
-    - 'dist/*'
-    - 'Cargo.lock'
-    - '.github/PULL_REQUEST_TEMPLATE.md'
+    - "dist/*"
+    - "Cargo.lock"
+    - "bindings/python/uv.lock"
+    - ".github/PULL_REQUEST_TEMPLATE.md"
   comment: on-failure

CONTRIBUTING.md

@@ -103,8 +103,8 @@ cargo 1.69.0 (6e9a83356 2023-04-12)
 
 Currently, iceberg-rust uses Docker to set up environment for integration tests. Native Docker has some limitations, please check (https://github.com/apache/iceberg-rust/pull/748). Please use Orbstack or Podman.
 
-For MacOS users, you can install [OrbStack as a docker alternative](reference/orbstack.md).
-For Podman users, refer to [Using Podman instead of Docker](reference/podman.md)
+For MacOS users, you can install [OrbStack as a docker alternative](website/src/reference/orbstack.md).
+For Podman users, refer to [Using Podman instead of Docker](website/src/reference/podman.md)
 
 ## Build