From e34ce563fcf77c620e0313d66d8e3cc521a97c3b Mon Sep 17 00:00:00 2001 From: Rong Ma Date: Mon, 26 Aug 2024 01:09:34 +0000 Subject: [PATCH] update comment --- cpp/velox/shuffle/VeloxSortShuffleWriter.cc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/cpp/velox/shuffle/VeloxSortShuffleWriter.cc b/cpp/velox/shuffle/VeloxSortShuffleWriter.cc index e3a357930d06..945e157f2238 100644 --- a/cpp/velox/shuffle/VeloxSortShuffleWriter.cc +++ b/cpp/velox/shuffle/VeloxSortShuffleWriter.cc @@ -104,8 +104,8 @@ arrow::Status VeloxSortShuffleWriter::init() { options_.partitioning == Partitioning::kSingle, arrow::Status::Invalid("VeloxSortShuffleWriter doesn't support single partition.")); allocateMinimalArray(); - // In Spark, sortedBuffer_ memory and compressionBuffer_ memory are pre-allocated and not counted into executor - // memory. To align with Spark, we use arrow::default_memory_pool() to avoid counting these memory in Gluten. + // In Spark, sortedBuffer_ memory and compressionBuffer_ memory are pre-allocated and counted into executor + // memory overhead. To align with Spark, we use arrow::default_memory_pool() to avoid counting these memory in Gluten. ARROW_ASSIGN_OR_RAISE( sortedBuffer_, arrow::AllocateBuffer(options_.compressionBufferSize, arrow::default_memory_pool())); rawBuffer_ = sortedBuffer_->mutable_data();