Skip to content

fix: track elapsed_compute for file stream - #18752

Closed
jizezhang wants to merge 1 commit into
apache:mainfrom
jizezhang:scan-time
Closed

jizezhang wants to merge 1 commit into
apache:mainfrom
jizezhang:scan-time

Conversation

@jizezhang

@jizezhang jizezhang commented Nov 16, 2025 •

Copy link
Copy Markdown
Contributor

Which issue does this PR close?

Address elapsed_compute baseline metrics not counting issue mentioned in #18195.

Before

> create external table customer 
stored as parquet
location '/Users/jizezhang/datafusion/benchmarks/data/tpch_sf1/customer';
0 row(s) fetched. 
Elapsed 0.026 seconds.

> explain analyze
select * from customer;
... elapsed_compute=8ns ...

After

> create external table customer 
stored as parquet
location '/Users/jizezhang/datafusion/benchmarks/data/tpch_sf1/customer';
0 row(s) fetched. 
Elapsed 0.033 seconds.

> explain analyze
select * from customer;
... elapsed_compute=1.786625ms ...

Rationale for this change

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

@github-actions github-actions Bot added the datasource Changes to the datasource crate label Nov 16, 2025
@jizezhang jizezhang closed this Nov 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

datasource Changes to the datasource crate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant