-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Pull Queries: Perf test #3545
Comments
@vinothchandar I believe you are already working on this? Maybe reassign? |
done. makes sense. |
Goals going forward :
|
Benchmark setup :Use ksql-datagen to generate some orders (more commandline opts to be added)
ksql queries to setup the final table. We decide how many orders we want in the table for pull queries (1000 in example below) and map each
Once we have this, the following lua script now generate pull queries that randomly picks an order in the range and queries it for benchmarking.
|
This is a flame graph of a pull query with the above mentioned fixes: The next bottleneck seems to be building the logical plan #3709 |
@vinothchandar can you add the specs of the environment you tested on? (cloud provider, instance type, memory, cpu specs, storage used, jvm settings) |
Cloud provider = AWS |
Current state:
Based on a simple setup like below
We found a few bottlenecks.
ServiceContext
keeps getting initialized per request + validation is performed per pull query as well (pull queries open a kafka admin client per request #3663) . Pull queries take about ~7ms averageThe text was updated successfully, but these errors were encountered: