Skip to content

Commit

Permalink
fix: total_bytes_written is not updated in celeborn partition writer
Browse files Browse the repository at this point in the history
  • Loading branch information
lgbo-ustc committed Aug 20, 2024
1 parent 9fcd488 commit 70909bf
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions cpp-ch/local-engine/Shuffle/PartitionWriter.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -469,6 +469,7 @@ size_t MemorySortCelebornPartitionWriter::evictPartitions()
celeborn_client->pushPartitionData(cur_partition_id, data.data(), data.size());
shuffle_writer->split_result.total_io_time += push_time_watch.elapsedNanoseconds();
shuffle_writer->split_result.partition_lengths[cur_partition_id] += data.size();
shuffle_writer->split_result.total_bytes_written += data.size();
}
output.restart();
};
Expand Down Expand Up @@ -586,6 +587,7 @@ size_t CelebornPartitionWriter::evictSinglePartition(size_t partition_id)
shuffle_writer->split_result.total_write_time += push_time_watch.elapsedNanoseconds();
shuffle_writer->split_result.total_io_time += push_time_watch.elapsedNanoseconds();
shuffle_writer->split_result.total_serialize_time += serialization_time_watch.elapsedNanoseconds();
shuffle_writer->split_result.total_bytes_written += written_bytes;
};

Stopwatch spill_time_watch;
Expand Down

0 comments on commit 70909bf

Please sign in to comment.