test(python): Fix test for new deltalake release

[ion-elgreco]

We need to explicitly provide overwrite mode when setting overwrite_schema = True

@stinodego :)

[stinodego]

@ion-elgreco Thanks! That fixes one of the tests, there is one more still failing - test_write_delta. It complains that a file already exists, probably because it's not overwriting correctly?

[ion-elgreco]

@stinodego ah didn't see that one, one sec

[stinodego]

Thanks! Good to merge if CI is green.

[ion-elgreco]

Thanks! Good to merge if CI is green.

Seems more tests are incompatible, will. Compile it locally and run it

[stinodego]

Seems more tests are incompatible, will. Compile it locally and run it

There seem to be some differences between Linux / Windows 🤔 first Windows failed and Linux passed, now it's the other way around.

[ion-elgreco]

Seems more tests are incompatible, will. Compile it locally and run it

There seem to be some differences between Linux / Windows 🤔 first Windows failed and Linux passed, now it's the other way around.

Ah yes that's because Linux wheels for 0.19 failed, due to some issue with maturin, I am waiting on my PR to be merged in delta-rs so I can re release them

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 80.29%. Comparing base (4157e92) to head (ddb4b2a).
Report is 10 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #18211      +/-   ##
==========================================
- Coverage   80.30%   80.29%   -0.01%     
==========================================
  Files        1498     1497       -1     
  Lines      198751   198741      -10     
  Branches     2833     2837       +4     
==========================================
- Hits       159598   159579      -19     
- Misses      38626    38635       +9     
  Partials      527      527              

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

[ion-elgreco]

@stinodego tests should be fine now, did a local compile with a deltalake v0.19. I'll ping you when I fixed our python wheels