From c383d9f16e5ae55bf49b67bdc837fca0088ad2d2 Mon Sep 17 00:00:00 2001 From: Piotr Rzysko Date: Thu, 28 Aug 2025 09:45:53 +0200 Subject: [PATCH] Remove outdated comment in order_by.rs The removed comment no longer matches the current code. The OrderByRemapping struct and the surrounding comments are sufficient to explain deduplication and remapping. --- core/translate/order_by.rs | 2 -- 1 file changed, 2 deletions(-) diff --git a/core/translate/order_by.rs b/core/translate/order_by.rs index a86a35ffb..129bc8d5b 100644 --- a/core/translate/order_by.rs +++ b/core/translate/order_by.rs @@ -321,8 +321,6 @@ pub struct OrderByRemapping { /// In case any of the ORDER BY sort keys are exactly equal to a result column, we can skip emitting that result column. /// If we skip a result column, we need to keep track what index in the ORDER BY sorter the result columns have, /// because the result columns should be emitted in the SELECT clause order, not the ORDER BY clause order. -/// -/// If any result columns can be skipped, this returns list of 2-tuples of (SkippedResultColumnIndex: usize, ResultColumnIndexInOrderBySorter: usize) pub fn order_by_deduplicate_result_columns( order_by: &[(Box, SortOrder)], result_columns: &[ResultSetColumn],