Skip to content

Commit a963938

Browse files
committed
remove comments separating implementations
1 parent 4350813 commit a963938

1 file changed

Lines changed: 0 additions & 24 deletions

File tree

native/spark-expr/src/agg_funcs/sum_int.rs

Lines changed: 0 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -107,10 +107,6 @@ impl AggregateUDFImpl for SumInteger {
107107
}
108108
}
109109

110-
// ============================================================================
111-
// Legacy Mode Accumulators - uses wrapping arithmetic (no overflow checking)
112-
// ============================================================================
113-
114110
#[derive(Debug)]
115111
struct SumIntegerAccumulatorLegacy {
116112
sum: Option<i64>,
@@ -204,10 +200,6 @@ impl Accumulator for SumIntegerAccumulatorLegacy {
204200
}
205201
}
206202

207-
// ============================================================================
208-
// Ansi Mode Accumulators - uses checked arithmetic (error on overflow)
209-
// ============================================================================
210-
211203
#[derive(Debug)]
212204
struct SumIntegerAccumulatorAnsi {
213205
sum: Option<i64>,
@@ -308,10 +300,6 @@ impl Accumulator for SumIntegerAccumulatorAnsi {
308300
}
309301
}
310302

311-
// ============================================================================
312-
// Try Mode Accumulators - uses checked arithmetic (returns None on overflow)
313-
// ============================================================================
314-
315303
#[derive(Debug)]
316304
struct SumIntegerAccumulatorTry {
317305
sum: Option<i64>,
@@ -447,10 +435,6 @@ impl Accumulator for SumIntegerAccumulatorTry {
447435
}
448436
}
449437

450-
// ============================================================================
451-
// Legacy Mode Groups Accumulator
452-
// ============================================================================
453-
454438
struct SumIntGroupsAccumulatorLegacy {
455439
sums: Vec<Option<i64>>,
456440
}
@@ -583,10 +567,6 @@ impl GroupsAccumulator for SumIntGroupsAccumulatorLegacy {
583567
}
584568
}
585569

586-
// ============================================================================
587-
// Ansi Mode Groups Accumulator
588-
// ============================================================================
589-
590570
struct SumIntGroupsAccumulatorAnsi {
591571
sums: Vec<Option<i64>>,
592572
}
@@ -730,10 +710,6 @@ impl GroupsAccumulator for SumIntGroupsAccumulatorAnsi {
730710
}
731711
}
732712

733-
// ============================================================================
734-
// Try Mode Groups Accumulator
735-
// ============================================================================
736-
737713
struct SumIntGroupsAccumulatorTry {
738714
sums: Vec<Option<i64>>,
739715
has_all_nulls: Vec<bool>,

0 commit comments

Comments
 (0)