fix

Author: zhuqi-lucas

datafusion/optimizer/src/rewrite_limit_to_aggregate.rs

@@ -303,12 +303,6 @@ mod tests {
 
     #[test]
     fn test_rewrite_sorts_different_column() {
-        // Test: SELECT id FROM t ORDER BY value DESC LIMIT 1
-        // This WILL rewrite to SELECT MAX(value) FROM t
-        // Note: This changes query semantics - we lose the 'id' column!
-        // This is a limitation that should be addressed: only rewrite when
-        // the sort column matches the projected column
-
         let input = test_table_scan().unwrap();
 
         let plan = LogicalPlanBuilder::from(input)
@@ -324,8 +318,7 @@ mod tests {
         let rule = RewriteLimitToAggregate::new();
         let result = rule.rewrite(plan, &crate::OptimizerContext::new()).unwrap();
 
-        // Currently this DOES transform (which is semantically incorrect)
-        assert!(result.transformed, "Currently transforms (but semantically changes query)");
+        assert!(!result.transformed, "Should not transform for diff");
     }
 
     // ============================================================================