Remove TODO about Replacing Sorting with Bucketing

Summary: We initially planned to improve the performance of the hard chunking stage by bucketing the streams by size (by most significant bit) instead of sorting them. However, after running benchmarks in D87571945. Sorting always performs better for large stream counts.

Differential Revision: D87573831

Author: macvincent

dwio/nimble/velox/VeloxWriter.cpp

@@ -1144,8 +1144,6 @@ bool VeloxWriter::evalauateFlushPolicy() {
     if (continueChunking) {
       // Relieve memory pressure by chunking small streams.
       // Sort streams for chunking based on raw memory usage.
-      // TODO(T240072104): Improve performance by bucketing the streams
-      // by size (by most significant bit) instead of sorting them.
       streamIndices.resize(streams.size());
       std::iota(streamIndices.begin(), streamIndices.end(), 0);
       std::sort(