Adapt comment

Dandandan · Dandandan · commit 771834ceaf20 · 2024-07-30T21:30:49.000+02:00
diff --git a/datafusion/functions-aggregate/src/approx_percentile_cont.rs b/datafusion/functions-aggregate/src/approx_percentile_cont.rs
@@ -394,7 +394,7 @@ impl Accumulator for ApproxPercentileAccumulator {
     }
 
     fn update_batch(&mut self, values: &[ArrayRef]) -> datafusion_common::Result<()> {
-        // respect nulls by default
+        // Remove any nulls before computing the percentile
         let mut values = Arc::clone(&values[0]);
         if values.nulls().is_some() {
             values = filter(&values, &is_not_null(&values)?)?;

Original file line number	Diff line number	Diff line change
`@@ -394,7 +394,7 @@ impl Accumulator for ApproxPercentileAccumulator {`
`394`	`394`	`}`
`395`	`395`
`396`	`396`	`fn update_batch(&mut self, values: &[ArrayRef]) -> datafusion_common::Result<()> {`
`397`		`- // respect nulls by default`
	`397`	`+ // Remove any nulls before computing the percentile`
`398`	`398`	`let mut values = Arc::clone(&values[0]);`
`399`	`399`	`if values.nulls().is_some() {`
`400`	`400`	`values = filter(&values, &is_not_null(&values)?)?;`