Dandandan commented on PR #16322:
URL: https://github.com/apache/datafusion/pull/16322#issuecomment-2954177103

   This seems really nice ๐Ÿš€ 
   On my machine I get roughly 10% improvement on queries with SPM - which I 
think makes sense on a 10 core machine (with less cores it might busy-poll).
   
   ```
   โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”ณโ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”ณโ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”ณโ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”“
   โ”ƒ Query        โ”ƒ       main โ”ƒ issue_16321 โ”ƒ        Change โ”ƒ
   โ”กโ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ•‡โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ•‡โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ•‡โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”ฉ
   โ”‚ QQuery 1     โ”‚  809.46 ms โ”‚   733.99 ms โ”‚ +1.10x faster โ”‚
   โ”‚ QQuery 2     โ”‚  165.80 ms โ”‚   148.89 ms โ”‚ +1.11x faster โ”‚
   โ”‚ QQuery 3     โ”‚  432.56 ms โ”‚   386.04 ms โ”‚ +1.12x faster โ”‚
   โ”‚ QQuery 4     โ”‚  459.88 ms โ”‚   467.66 ms โ”‚     no change โ”‚
   โ”‚ QQuery 5     โ”‚  671.13 ms โ”‚   621.34 ms โ”‚ +1.08x faster โ”‚
   โ”‚ QQuery 6     โ”‚  181.36 ms โ”‚   184.77 ms โ”‚     no change โ”‚
   โ”‚ QQuery 7     โ”‚  959.96 ms โ”‚   871.05 ms โ”‚ +1.10x faster โ”‚
   โ”‚ QQuery 8     โ”‚  672.39 ms โ”‚   627.96 ms โ”‚ +1.07x faster โ”‚
   โ”‚ QQuery 9     โ”‚ 1101.98 ms โ”‚  1023.84 ms โ”‚ +1.08x faster โ”‚
   โ”‚ QQuery 10    โ”‚  638.41 ms โ”‚   552.55 ms โ”‚ +1.16x faster โ”‚
   โ”‚ QQuery 11    โ”‚  126.22 ms โ”‚   123.91 ms โ”‚     no change โ”‚
   โ”‚ QQuery 12    โ”‚  358.25 ms โ”‚   329.10 ms โ”‚ +1.09x faster โ”‚
   โ”‚ QQuery 13    โ”‚  720.99 ms โ”‚   656.95 ms โ”‚ +1.10x faster โ”‚
   โ”‚ QQuery 14    โ”‚  247.23 ms โ”‚   251.52 ms โ”‚     no change โ”‚
   โ”‚ QQuery 15    โ”‚  395.58 ms โ”‚   369.79 ms โ”‚ +1.07x faster โ”‚
   โ”‚ QQuery 16    โ”‚  110.37 ms โ”‚   103.96 ms โ”‚ +1.06x faster โ”‚
   โ”‚ QQuery 17    โ”‚ 1193.78 ms โ”‚  1206.00 ms โ”‚     no change โ”‚
   โ”‚ QQuery 18    โ”‚ 1846.58 ms โ”‚  1668.23 ms โ”‚ +1.11x faster โ”‚
   โ”‚ QQuery 19    โ”‚  412.76 ms โ”‚   400.33 ms โ”‚     no change โ”‚
   โ”‚ QQuery 20    โ”‚  421.08 ms โ”‚   392.58 ms โ”‚ +1.07x faster โ”‚
   โ”‚ QQuery 21    โ”‚ 1363.39 ms โ”‚  1275.33 ms โ”‚ +1.07x faster โ”‚
   โ”‚ QQuery 22    โ”‚  149.67 ms โ”‚   132.31 ms โ”‚ +1.13x faster โ”‚
   โ””โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”ดโ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”ดโ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”ดโ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”˜
   โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”ณโ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”“
   โ”ƒ Benchmark Summary          โ”ƒ            โ”ƒ
   โ”กโ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ•‡โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”ฉ
   โ”‚ Total Time (main)          โ”‚ 13438.84ms โ”‚
   โ”‚ Total Time (issue_16321)   โ”‚ 12528.08ms โ”‚
   โ”‚ Average Time (main)        โ”‚   610.86ms โ”‚
   โ”‚ Average Time (issue_16321) โ”‚   569.46ms โ”‚
   โ”‚ Queries Faster             โ”‚         16 โ”‚
   โ”‚ Queries Slower             โ”‚          0 โ”‚
   โ”‚ Queries with No Change     โ”‚          6 โ”‚
   โ”‚ Queries with Failure       โ”‚          0 โ”‚
   โ””โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”ดโ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”€โ”˜
   
   ```


-- 
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