Dandandan commented on PR #7932:
URL: 
https://github.com/apache/arrow-datafusion/pull/7932#issuecomment-1837445283

   Ok, on a flight I got some more time to find the regression. It was related 
to join selection and statistics.
   
   Current version shows no regressions anymore ๐ŸŽ‰
   
   ```
   --------------------
   Benchmark tpch.json
   --------------------
   โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”ณโ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”ณโ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”ณโ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”“
   โ”ƒ Query        โ”ƒ     main โ”ƒ projection_filter_exec โ”ƒ        Change โ”ƒ
   โ”กโ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ•‡โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ•‡โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ•‡โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”ฉ
   โ”‚ QQuery 1     โ”‚ 151.50ms โ”‚               151.57ms โ”‚     no change โ”‚
   โ”‚ QQuery 2     โ”‚  49.60ms โ”‚                49.71ms โ”‚     no change โ”‚
   โ”‚ QQuery 3     โ”‚  70.18ms โ”‚                68.88ms โ”‚     no change โ”‚
   โ”‚ QQuery 4     โ”‚  51.75ms โ”‚                50.40ms โ”‚     no change โ”‚
   โ”‚ QQuery 5     โ”‚  82.02ms โ”‚                81.63ms โ”‚     no change โ”‚
   โ”‚ QQuery 6     โ”‚  31.84ms โ”‚                31.47ms โ”‚     no change โ”‚
   โ”‚ QQuery 7     โ”‚ 102.68ms โ”‚               102.40ms โ”‚     no change โ”‚
   โ”‚ QQuery 8     โ”‚ 107.65ms โ”‚               107.11ms โ”‚     no change โ”‚
   โ”‚ QQuery 9     โ”‚ 131.21ms โ”‚               130.31ms โ”‚     no change โ”‚
   โ”‚ QQuery 10    โ”‚ 137.28ms โ”‚               134.48ms โ”‚     no change โ”‚
   โ”‚ QQuery 11    โ”‚  37.14ms โ”‚                36.55ms โ”‚     no change โ”‚
   โ”‚ QQuery 12    โ”‚  85.27ms โ”‚                84.92ms โ”‚     no change โ”‚
   โ”‚ QQuery 13    โ”‚ 190.22ms โ”‚               180.37ms โ”‚ +1.05x faster โ”‚
   โ”‚ QQuery 14    โ”‚  50.64ms โ”‚                51.16ms โ”‚     no change โ”‚
   โ”‚ QQuery 15    โ”‚  61.33ms โ”‚                55.79ms โ”‚ +1.10x faster โ”‚
   โ”‚ QQuery 16    โ”‚  54.56ms โ”‚                52.42ms โ”‚     no change โ”‚
   โ”‚ QQuery 17    โ”‚ 100.46ms โ”‚               102.43ms โ”‚     no change โ”‚
   โ”‚ QQuery 18    โ”‚ 188.07ms โ”‚               188.75ms โ”‚     no change โ”‚
   โ”‚ QQuery 19    โ”‚  98.04ms โ”‚                98.17ms โ”‚     no change โ”‚
   โ”‚ QQuery 20    โ”‚  55.19ms โ”‚                53.82ms โ”‚     no change โ”‚
   โ”‚ QQuery 21    โ”‚ 128.41ms โ”‚               120.85ms โ”‚ +1.06x faster โ”‚
   โ”‚ QQuery 22    โ”‚  38.39ms โ”‚                37.76ms โ”‚     no change โ”‚
   โ””โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”ดโ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”ดโ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”ดโ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”˜
   --------------------
   Benchmark tpch_mem.json
   --------------------
   โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”ณโ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”ณโ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”ณโ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”“
   โ”ƒ Query        โ”ƒ     main โ”ƒ projection_filter_exec โ”ƒ        Change โ”ƒ
   โ”กโ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ•‡โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ•‡โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ•‡โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”ฉ
   โ”‚ QQuery 1     โ”‚  91.51ms โ”‚                89.51ms โ”‚     no change โ”‚
   โ”‚ QQuery 2     โ”‚  25.79ms โ”‚                22.87ms โ”‚ +1.13x faster โ”‚
   โ”‚ QQuery 3     โ”‚  53.71ms โ”‚                41.13ms โ”‚ +1.31x faster โ”‚
   โ”‚ QQuery 4     โ”‚  51.74ms โ”‚                28.65ms โ”‚ +1.81x faster โ”‚
   โ”‚ QQuery 5     โ”‚ 115.24ms โ”‚                84.50ms โ”‚ +1.36x faster โ”‚
   โ”‚ QQuery 6     โ”‚   9.28ms โ”‚                 8.36ms โ”‚ +1.11x faster โ”‚
   โ”‚ QQuery 7     โ”‚ 215.76ms โ”‚               212.17ms โ”‚     no change โ”‚
   โ”‚ QQuery 8     โ”‚  59.93ms โ”‚                60.06ms โ”‚     no change โ”‚
   โ”‚ QQuery 9     โ”‚  58.50ms โ”‚                57.97ms โ”‚     no change โ”‚
   โ”‚ QQuery 10    โ”‚ 115.36ms โ”‚                79.37ms โ”‚ +1.45x faster โ”‚
   โ”‚ QQuery 11    โ”‚  19.34ms โ”‚                19.77ms โ”‚     no change โ”‚
   โ”‚ QQuery 12    โ”‚  59.47ms โ”‚                36.31ms โ”‚ +1.64x faster โ”‚
   โ”‚ QQuery 13    โ”‚  54.12ms โ”‚                43.69ms โ”‚ +1.24x faster โ”‚
   โ”‚ QQuery 14    โ”‚  18.04ms โ”‚                12.17ms โ”‚ +1.48x faster โ”‚
   โ”‚ QQuery 15    โ”‚  58.53ms โ”‚                38.91ms โ”‚ +1.50x faster โ”‚
   โ”‚ QQuery 16    โ”‚  21.86ms โ”‚                21.94ms โ”‚     no change โ”‚
   โ”‚ QQuery 17    โ”‚  53.52ms โ”‚                53.08ms โ”‚     no change โ”‚
   โ”‚ QQuery 18    โ”‚ 156.56ms โ”‚               145.73ms โ”‚ +1.07x faster โ”‚
   โ”‚ QQuery 19    โ”‚  35.22ms โ”‚                28.24ms โ”‚ +1.25x faster โ”‚
   โ”‚ QQuery 20    โ”‚  63.21ms โ”‚                50.03ms โ”‚ +1.26x faster โ”‚
   โ”‚ QQuery 21    โ”‚ 249.52ms โ”‚               173.98ms โ”‚ +1.43x faster โ”‚
   โ”‚ QQuery 22    โ”‚  14.70ms โ”‚                14.32ms โ”‚     no change โ”‚
   โ””โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”ดโ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”ดโ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”ดโ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”˜
   ```


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to