Skip to content

Commit

Permalink
remove unneed df config
Browse files Browse the repository at this point in the history
  • Loading branch information
Blajda committed Dec 17, 2023
1 parent a5ecd95 commit 51a4d70
Showing 1 changed file with 1 addition and 4 deletions.
5 changes: 1 addition & 4 deletions crates/deltalake-core/src/operations/merge/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -1172,10 +1172,7 @@ impl std::future::IntoFuture for MergeBuilder {
let state = this.state.unwrap_or_else(|| {
//TODO: Datafusion's Hashjoin has some memory issues. Running with all cores results in a OoM. Can be removed when upstream improvemetns are made.
let config: SessionConfig = DeltaSessionConfig::default().into();
let config = config
.with_target_partitions(1)
//TODO: Datafusion physical optimizer has a bug that inserts a coalesece in a bad spot
.with_coalesce_batches(true);
let config = config.with_target_partitions(1);
let session = SessionContext::new_with_config(config);

// If a user provides their own their DF state then they must register the store themselves
Expand Down

0 comments on commit 51a4d70

Please sign in to comment.