Revert "[VL] Sort by buffer size before evict partition buffers minSize (apache#10009)"

zml1206 · zml1206 · commit 88cdfd819950 · 2026-06-10T10:22:55.000+08:00
This reverts commit 31ff79f.
diff --git a/cpp/velox/shuffle/VeloxHashShuffleWriter.cc b/cpp/velox/shuffle/VeloxHashShuffleWriter.cc
@@ -1388,7 +1388,6 @@ arrow::Result<int64_t> VeloxHashShuffleWriter::evictPartitionBuffersMinSize(int6
     }
     pidToSize.emplace_back(pid, partitionBufferSize_[pid]);
   }
-  std::sort(pidToSize.begin(), pidToSize.end(), [&](const auto& a, const auto& b) { return a.second > b.second; });
   if (!pidToSize.empty()) {
     for (auto& item : pidToSize) {
       auto pid = item.first;

Original file line number	Diff line number	Diff line change
`@@ -1388,7 +1388,6 @@ arrow::Result<int64_t> VeloxHashShuffleWriter::evictPartitionBuffersMinSize(int6`
`1388`	`1388`	`}`
`1389`	`1389`	`pidToSize.emplace_back(pid, partitionBufferSize_[pid]);`
`1390`	`1390`	`}`
`1391`		`- std::sort(pidToSize.begin(), pidToSize.end(), [&](const auto& a, const auto& b) { return a.second > b.second; });`
`1392`	`1391`	`if (!pidToSize.empty()) {`
`1393`	`1392`	`for (auto& item : pidToSize) {`
`1394`	`1393`	`auto pid = item.first;`