This is an automated email from the ASF dual-hosted git repository.

dheres pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/datafusion.git


The following commit(s) were added to refs/heads/main by this push:
     new 70aba2bd6c minor: always time batch_filter even when the result is an 
empty batch (#11775)
70aba2bd6c is described below

commit 70aba2bd6c0351f780157f2595a51ac4a9df56e2
Author: Andy Grove <[email protected]>
AuthorDate: Fri Aug 2 00:11:59 2024 -0600

    minor: always time batch_filter even when the result is an empty batch 
(#11775)
---
 datafusion/physical-plan/src/filter.rs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/datafusion/physical-plan/src/filter.rs 
b/datafusion/physical-plan/src/filter.rs
index 67de098964..69bcfefcd4 100644
--- a/datafusion/physical-plan/src/filter.rs
+++ b/datafusion/physical-plan/src/filter.rs
@@ -380,11 +380,11 @@ impl Stream for FilterExecStream {
                 Some(Ok(batch)) => {
                     let timer = 
self.baseline_metrics.elapsed_compute().timer();
                     let filtered_batch = batch_filter(&batch, 
&self.predicate)?;
+                    timer.done();
                     // skip entirely filtered batches
                     if filtered_batch.num_rows() == 0 {
                         continue;
                     }
-                    timer.done();
                     poll = Poll::Ready(Some(Ok(filtered_batch)));
                     break;
                 }


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to