zhuqi-lucas commented on PR #16690:
URL: https://github.com/apache/datafusion/pull/16690#issuecomment-3048816186

   > ๐Ÿค–: Benchmark completed
   > 
   > Details
   > 
   > ```
   > Comparing HEAD and alamb_update_arrow_56.0.0
   > --------------------
   > Benchmark sort_tpch1.json
   > --------------------
   > โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”ณโ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”ณโ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”ณโ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”“
   > โ”ƒ Query        โ”ƒ       HEAD โ”ƒ alamb_update_arrow_56.0 โ”ƒ       Change โ”ƒ
   > โ”กโ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ•‡โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ•‡โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ•‡โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”ฉ
   > โ”‚ Q1           โ”‚  330.29 ms โ”‚               350.13 ms โ”‚ 1.06x slower โ”‚
   > โ”‚ Q2           โ”‚  308.58 ms โ”‚               312.59 ms โ”‚    no change โ”‚
   > โ”‚ Q3           โ”‚ 1051.46 ms โ”‚              1076.53 ms โ”‚    no change โ”‚
   > โ”‚ Q4           โ”‚  444.92 ms โ”‚               462.29 ms โ”‚    no change โ”‚
   > โ”‚ Q5           โ”‚  410.39 ms โ”‚               412.36 ms โ”‚    no change โ”‚
   > โ”‚ Q6           โ”‚  449.16 ms โ”‚               452.76 ms โ”‚    no change โ”‚
   > โ”‚ Q7           โ”‚  783.43 ms โ”‚               833.85 ms โ”‚ 1.06x slower โ”‚
   > โ”‚ Q8           โ”‚  696.74 ms โ”‚               712.94 ms โ”‚    no change โ”‚
   > โ”‚ Q9           โ”‚  732.56 ms โ”‚               732.64 ms โ”‚    no change โ”‚
   > โ”‚ Q10          โ”‚ 1077.29 ms โ”‚              1090.28 ms โ”‚    no change โ”‚
   > โ”‚ Q11          โ”‚  547.35 ms โ”‚               563.97 ms โ”‚    no change โ”‚
   > โ””โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”ดโ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”ดโ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”ดโ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”˜
   > โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”ณโ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”“
   > โ”ƒ Benchmark Summary                      โ”ƒ           โ”ƒ
   > โ”กโ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ•‡โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”ฉ
   > โ”‚ Total Time (HEAD)                      โ”‚ 6832.16ms โ”‚
   > โ”‚ Total Time (alamb_update_arrow_56.0)   โ”‚ 7000.33ms โ”‚
   > โ”‚ Average Time (HEAD)                    โ”‚  621.11ms โ”‚
   > โ”‚ Average Time (alamb_update_arrow_56.0) โ”‚  636.39ms โ”‚
   > โ”‚ Queries Faster                         โ”‚         0 โ”‚
   > โ”‚ Queries Slower                         โ”‚         2 โ”‚
   > โ”‚ Queries with No Change                 โ”‚         9 โ”‚
   > โ”‚ Queries with Failure                   โ”‚         0 โ”‚
   > โ””โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”ดโ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”˜
   > ```
   
   Thank you @alamb , it seems no improvement for sort dependencies(So it means 
the merge will occupied most time), so we gain most for the ported PRs to merge 
phase from:
   
   https://github.com/apache/datafusion/pull/16509
   https://github.com/apache/datafusion/pull/16630
   
   
   
   


-- 
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-unsubscr...@datafusion.apache.org

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


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

Reply via email to