Skip to content

Conversation

@zhuqi-lucas
Copy link
Contributor

Which issue does this PR close?

Minor: add average time for clickbench benchmark query

Rationale for this change

Minor: add average time for clickbench benchmark query

What changes are included in this PR?

Minor: add average time for clickbench benchmark query

Are these changes tested?

Yes

Are there any user-facing changes?

Minor: add average time for clickbench benchmark query

@zhuqi-lucas
Copy link
Contributor Author

Testing result:

cargo run --profile release-nonlto  --bin dfbench -- clickbench     -q 1
    Finished `release-nonlto` profile [optimized] target(s) in 0.27s
     Running `target/release-nonlto/dfbench clickbench -q 1`
Running benchmarks with the following options: RunOpt { query: Some(1), common: CommonOpt { iterations: 3, partitions: None, batch_size: 8192, mem_pool_type: "fair", memory_limit: None, sort_spill_reservation_bytes: None, debug: false }, path: "benchmarks/data/hits.parquet", queries_path: "benchmarks/queries/clickbench/queries.sql", output_path: None }
Q1: SELECT COUNT(*) FROM hits WHERE "AdvEngineID" <> 0;
Query 1 iteration 0 took 123.9 ms and returned 1 rows
Query 1 iteration 1 took 56.0 ms and returned 1 rows
Query 1 iteration 2 took 47.3 ms and returned 1 rows
Query 1 avg time: 75.75 ms

Copy link
Contributor

@2010YOUY01 2010YOUY01 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thank you.

Copy link
Member

@xudong963 xudong963 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you

@xudong963 xudong963 merged commit 3f538f8 into apache:main Mar 24, 2025
27 checks passed
qstommyshu pushed a commit to qstommyshu/datafusion that referenced this pull request Mar 27, 2025
nirnayroy pushed a commit to nirnayroy/datafusion that referenced this pull request May 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants