Skip to content

Conversation

crcarreras
Copy link

Added code to execute benchmarks on Databricks SQL, and Databricks SQL compatible FireScale queries.

Added code to execute benchmarks on Databricks SQL, and Databricks SQL compatible FireScale queries.
@crcarreras crcarreras requested a review from fb-cole July 8, 2025 20:27
Copy link
Collaborator

@fb-cole fb-cole left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should also make sure we have a warmup.sql file for Databricks? Or an empty one if they're not doing caching.

Comment on lines +86 to +96
VACUUM uservisits;

VACUUM uservisits;

VACUUM rankings;

VACUUM searchwords;

VACUUM agents;

VACUUM ipaddresses;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you VACUUM in Databricks / have you validated that this setup file works for Databricks?

@@ -0,0 +1,52 @@
{
"query 1":
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file isn't right. It should be the queries we selected for concurrent testing with K6.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants