Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FEAT] Allow overwrite / overwrite partitions for write operations #1986

Closed
wants to merge 6 commits into from

Conversation

colin-ho
Copy link
Contributor

@colin-ho colin-ho commented Mar 7, 2024

Closes #1768

This is a POC for adding overwrite / overwrite partitions mode for our write methods. The idea is to collect all the file paths that were written across partitions into a single partition via a reduce-merge step, then performing an ls to get all the file paths in the root dir, then deleting all files that were not written as part of the write operation.

@github-actions github-actions bot added the enhancement New feature or request label Mar 7, 2024
Copy link

codecov bot commented Mar 7, 2024

Codecov Report

Attention: Patch coverage is 96.29630% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 84.72%. Comparing base (ce8d678) to head (95f5d39).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1986      +/-   ##
==========================================
+ Coverage   84.67%   84.72%   +0.04%     
==========================================
  Files          57       57              
  Lines        6317     6343      +26     
==========================================
+ Hits         5349     5374      +25     
- Misses        968      969       +1     
Files Coverage Δ
daft/execution/execution_step.py 93.73% <100.00%> (+0.17%) ⬆️
daft/execution/rust_physical_plan_shim.py 98.52% <100.00%> (+0.04%) ⬆️
daft/filesystem.py 62.87% <92.30%> (+2.22%) ⬆️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEAT] Allow for selection of append/overwrite/overwrite_partitions options when writing data
1 participant