From 559fcd0ff5af49671bcbdcf324bd9911b08bb4c8 Mon Sep 17 00:00:00 2001 From: Abhi Agarwal Date: Mon, 20 Oct 2025 15:19:08 -0400 Subject: [PATCH 1/3] Reduce wheel size Signed-off-by: Abhi Agarwal --- .github/workflows/python_release.yml | 2 -- Cargo.toml | 15 +++++++++++---- python/Cargo.toml | 5 ----- python/pyproject.toml | 1 + 4 files changed, 12 insertions(+), 11 deletions(-) diff --git a/.github/workflows/python_release.yml b/.github/workflows/python_release.yml index 9a71871aa9..35c147c14d 100644 --- a/.github/workflows/python_release.yml +++ b/.github/workflows/python_release.yml @@ -83,7 +83,6 @@ jobs: env: MATURIN_PYPI_TOKEN: ${{ secrets.PYPI_TOKEN }} with: - maturin-version: v1.6.0 # https://github.com/PyO3/maturin/issues/2154 target: x86_64-unknown-linux-gnu command: publish args: --skip-existing -m python/Cargo.toml @@ -149,7 +148,6 @@ jobs: env: MATURIN_PYPI_TOKEN: ${{ secrets.PYPI_TOKEN }} with: - maturin-version: v1.6.0 # https://github.com/PyO3/maturin/issues/2154 target: x86_64-unknown-linux-musl command: publish args: --skip-existing -m python/Cargo.toml --zig diff --git a/Cargo.toml b/Cargo.toml index 7fc95e9d9e..082478c63c 100644 --- a/Cargo.toml +++ b/Cargo.toml @@ -15,10 +15,6 @@ license = "Apache-2.0" documentation = "https://docs.rs/deltalake" repository = "https://github.com/delta-io/delta.rs" -[profile.release-with-debug] -inherits = "release" -debug = true - # Reducing the debuginfo for the test profile in order to trim the disk and RAM # usage during development # Date: Fri, 24 Oct 2025 21:35:12 -0400 Subject: [PATCH 2/3] Add warmup to benchmarks Signed-off-by: Abhi Agarwal --- python/tests/test_benchmark.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/python/tests/test_benchmark.py b/python/tests/test_benchmark.py index b59fcb6ab7..215fb6f4b0 100644 --- a/python/tests/test_benchmark.py +++ b/python/tests/test_benchmark.py @@ -40,7 +40,7 @@ def setup() -> None: def func(table_path: str) -> None: write_deltalake(table_path, sample_table) - benchmark.pedantic(func, setup=setup, rounds=5) + benchmark.pedantic(func, setup=setup, rounds=5, warmup_rounds=3) # TODO: figure out why this assert is failing # dt = DeltaTable(str(tmp_path)) @@ -70,7 +70,7 @@ def setup() -> None: def func(table_path: str) -> None: write_deltalake(table_path, sample_table, storage_options=storage_options) - benchmark.pedantic(func, setup=setup, rounds=5) + benchmark.pedantic(func, setup=setup, rounds=5, warmup_rounds=3) @pytest.mark.pyarrow @@ -146,7 +146,7 @@ def func(dt, max_concurrent_tasks): ) # We need to recreate the table for each benchmark run - results = benchmark.pedantic(func, setup=setup, rounds=5) + results = benchmark.pedantic(func, setup=setup, rounds=5, warmup_rounds=3) assert results["numFilesRemoved"] == 50 assert results["numFilesAdded"] == 5 @@ -207,7 +207,7 @@ def func(dt, max_concurrent_tasks): max_concurrent_tasks=max_concurrent_tasks, target_size=1024 * 1024 * 1024 ) - results = benchmark.pedantic(func, setup=setup, rounds=5) + results = benchmark.pedantic(func, setup=setup, rounds=5, warmup_rounds=3) assert results["numFilesRemoved"] == 50 assert results["numFilesAdded"] == 5 From 76bc3e0179468285ccbaf3d50fce52c76d5c139a Mon Sep 17 00:00:00 2001 From: Abhi Agarwal Date: Fri, 24 Oct 2025 21:35:18 -0400 Subject: [PATCH 3/3] Change to opt-level 3 Signed-off-by: Abhi Agarwal --- Cargo.toml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Cargo.toml b/Cargo.toml index 082478c63c..0c54f9cfaa 100644 --- a/Cargo.toml +++ b/Cargo.toml @@ -113,7 +113,7 @@ debug = true [profile.release-python] inherits = "release" -opt-level = "s" +opt-level = 3 codegen-units = 1 lto = "fat" strip = true