Skip to content

Commit

Permalink
[GLUTEN-5724][VL] Remove redundant counter for calculating VeloxShuff…
Browse files Browse the repository at this point in the history
…leWriter spill time (#5725)
  • Loading branch information
marin-ma authored May 14, 2024
1 parent 33f9935 commit 182a029
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions cpp/core/shuffle/LocalPartitionWriter.cc
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,6 @@ class LocalPartitionWriter::LocalSpiller {
codec_(codec) {}

arrow::Status spill(uint32_t partitionId, std::unique_ptr<BlockPayload> payload) {
ScopedTimer timer(&spillTime_);
// Check spill Type.
if (payload->type() != Payload::kUncompressed) {
return arrow::Status::Invalid(
Expand Down Expand Up @@ -317,8 +316,6 @@ class LocalPartitionWriter::PayloadCache {

arrow::Result<std::shared_ptr<Spill>>
spill(const std::string& spillFile, arrow::MemoryPool* pool, arrow::util::Codec* codec) {
ScopedTimer timer(&spillTime_);

std::shared_ptr<Spill> diskSpill = nullptr;
ARROW_ASSIGN_OR_RAISE(auto os, arrow::io::FileOutputStream::Open(spillFile, true));
ARROW_ASSIGN_OR_RAISE(auto start, os->Tell());
Expand Down

0 comments on commit 182a029

Please sign in to comment.