Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Increase split loader concurrency to 64 #9979

Merged

Conversation

sopel39
Copy link
Member

@sopel39 sopel39 commented Nov 17, 2021

sf1000 partitioned orc

	concurrency 	TPCH wall time 	TPC-DS wall time 	TPCH CPU time 	TPC-DS CPU time
0 	4	 	930.229833	1263.213167	 	96815.4 	127909.085000
1 	64	 	904.222000 	1164.819667(8%) 	94923.5 	127906.049667
2 	32	 	934.815000 	1260.967000	 	96956.3 	128975.193333

64 yields best results
Some individual queries see 2x, 3x improvment

@sopel39 sopel39 requested review from martint and losipiuk November 17, 2021 12:57
sf1000 orc partitioned results
	concurrency 	TPCH wall time 	TPC-DS wall time 	TPCH CPU time 	TPC-DS CPU time
0 	4	 	930.229833	1263.213167	 	96815.4 	127909.085000
1 	64	 	904.222000 	1164.819667	 	94923.5 	127906.049667
2 	32	 	934.815000 	1260.967000	 	96956.3 	128975.193333
@sopel39 sopel39 force-pushed the ks/increase_loader_concurrency branch from 70880b5 to 3dea758 Compare November 17, 2021 12:59
@cla-bot cla-bot bot added the cla-signed label Nov 17, 2021
@sopel39 sopel39 merged commit 5697349 into trinodb:master Nov 17, 2021
@sopel39 sopel39 mentioned this pull request Nov 17, 2021
12 tasks
@github-actions github-actions bot added this to the 365 milestone Nov 17, 2021
@sopel39
Copy link
Member Author

sopel39 commented Nov 18, 2021

cc @findepi @losipiuk

@sopel39 sopel39 deleted the ks/increase_loader_concurrency branch November 22, 2021 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants