Skip to content

Commit

Permalink
fix: Delta update reader lifecycle issue
Browse files Browse the repository at this point in the history
Summary:
In ScanSpec we only clear the column updater for root but not the
actual columns, this results in lifecycle issue when a non-Metalake split is
following a Makelake split.

Differential Revision: D66402400
  • Loading branch information
Yuhta authored and facebook-github-bot committed Nov 23, 2024
1 parent bf3fba7 commit cc823f3
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion velox/dwio/common/ScanSpec.h
Original file line number Diff line number Diff line change
Expand Up @@ -346,7 +346,7 @@ class ScanSpec {
for (auto& child : children_) {
// Only top level columns can have delta updates.
if (child->deltaUpdate_) {
setDeltaUpdate(nullptr);
child->setDeltaUpdate(nullptr);
}
}
}
Expand Down

0 comments on commit cc823f3

Please sign in to comment.