Re: [PR] Simplify sqllogictest timing summary to boolean flag and remove top-N modes [datafusion]

["alamb (via GitHub)" <gi...@apache.org>]

alamb commented on PR #20598:
URL: https://github.com/apache/datafusion/pull/20598#issuecomment-4120959934

   I tried it out locally
   
   ```shell
   cargo test --profile=ci --test sqllogictests -- --timing-summary | head -n 10
   ```
   
   And it looks good to me:
   
   ```
   Per-file elapsed summary (deterministic):
     1.    3.960s  joins.slt
     2.    3.837s  aggregate.slt
     3.    3.604s  imdb.slt
     4.    3.507s  push_down_filter_regression.slt
     5.    3.421s  aggregate_skip_partial.slt
     6.    3.143s  array.slt
     7.    2.652s  window.slt
     8.    2.461s  group_by.slt
     9.    1.589s  clickbench.slt
   Completed 415 test files in 4 seconds
   ```
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For additional commands, e-mail: github-help@datafusion.apache.org