This is an automated email from the ASF dual-hosted git repository.
changchen pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/incubator-gluten.git
The following commit(s) were added to refs/heads/main by this push:
new 79b7f29953 [GLUTEN-7427][CH]Revert "fix (#7349)"
79b7f29953 is described below
commit 79b7f299531177e2c87783d4d6d264b59db57574
Author: Chang chen <[email protected]>
AuthorDate: Tue Oct 8 13:11:17 2024 +0800
[GLUTEN-7427][CH]Revert "fix (#7349)"
This reverts commit dc8a91ce337a2e8f20bbee12a38ca946ebdff06f.
---
cpp-ch/local-engine/Storages/MergeTree/SparkStorageMergeTree.cpp | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/cpp-ch/local-engine/Storages/MergeTree/SparkStorageMergeTree.cpp
b/cpp-ch/local-engine/Storages/MergeTree/SparkStorageMergeTree.cpp
index e1b3c1d073..7f1c5e4997 100644
--- a/cpp-ch/local-engine/Storages/MergeTree/SparkStorageMergeTree.cpp
+++ b/cpp-ch/local-engine/Storages/MergeTree/SparkStorageMergeTree.cpp
@@ -176,8 +176,6 @@ std::vector<MergeTreeDataPartPtr>
SparkStorageMergeTree::loadDataPartsWithNames(
data_parts.emplace_back(res.part);
}
- // without it "test mergetree optimize partitioned by one low card column"
will log ERROR
- calculateColumnAndSecondaryIndexSizesImpl();
return data_parts;
}
@@ -252,6 +250,9 @@ MergeTreeData::LoadPartResult
SparkStorageMergeTree::loadDataPart(
if (res.part->hasLightweightDelete())
has_lightweight_delete_parts.store(true);
+ // without it "test mergetree optimize partitioned by one low card column"
will log ERROR
+ calculateColumnAndSecondaryIndexSizesImpl();
+
LOG_TRACE(log, "Finished loading {} part {} on disk {}",
magic_enum::enum_name(to_state), part_name, part_disk_ptr->getName());
return res;
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
