Skip to content

chore: tighten duplicate argument validation in create() and vacuum()#4299

Merged
ion-elgreco merged 1 commit intodelta-io:mainfrom
ethan-tyler:fix/4283-python-dup-shim
Mar 20, 2026
Merged

chore: tighten duplicate argument validation in create() and vacuum()#4299
ion-elgreco merged 1 commit intodelta-io:mainfrom
ethan-tyler:fix/4283-python-dup-shim

Conversation

@ethan-tyler
Copy link
Copy Markdown
Collaborator

Description

Tighten the Python compatibility handling in DeltaTable.create() and DeltaTable.vacuum() ensuring duplicate values are rejected when legacy positional arguments are mixed with keywords.

Related Issue(s)

Documentation

Signed-off-by: Ethan Urbanski <ethan@urbanskitech.com>
@github-actions github-actions Bot added the binding/python Issues for the Python package label Mar 19, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 19, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 77.15%. Comparing base (c85075e) to head (6819c5a).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #4299   +/-   ##
=======================================
  Coverage   77.15%   77.15%           
=======================================
  Files         167      167           
  Lines       50266    50266           
  Branches    50266    50266           
=======================================
  Hits        38781    38781           
  Misses       9515     9515           
  Partials     1970     1970           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ion-elgreco
Copy link
Copy Markdown
Collaborator

Not a great fan of these changes tbh, so lets rip the bandaid off as soon as possible in a new minor release

@ion-elgreco ion-elgreco merged commit 481fdde into delta-io:main Mar 20, 2026
30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

binding/python Issues for the Python package

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants