-
Notifications
You must be signed in to change notification settings - Fork 169
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
chore: Add documentation on running benchmarks with Microk8s #848
Conversation
--jars local://$COMET_JAR \ | ||
--conf spark.executor.extraClassPath=$COMET_JAR \ | ||
--conf spark.driver.extraClassPath=$COMET_JAR \ | ||
--conf spark.plugins=org.apache.spark.CometPlugin \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
--name comet-tpcbench \ | ||
--driver-memory 8G \ | ||
--conf spark.driver.memory=8G \ | ||
--conf spark.executor.instances=1 \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
for benches its probably good to run it in scale? to put instances > 1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is really meant as an example. I would expect users to customize the script for their local environment.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm thanks @andygrove. some minor questions
Co-authored-by: Oleks V <comphead@users.noreply.github.com>
@andygrove are you planning to merge it, I'd also like to add the Helm chart section to run in the Kubernetes cluster |
Which issue does this PR close?
Part of #602
Rationale for this change
What changes are included in this PR?
How are these changes tested?