Add warmup run #11
Add warmup run #11
Conversation
bench.py
Outdated
| query_type, | ||
| ingest, | ||
| index, | ||
| prefix + "-warmup", |
There was a problem hiding this comment.
I don't think we need to create and ingest a whole new set of tables
There was a problem hiding this comment.
Make sense! Change to warmup with query only
| @@ -86,6 +86,14 @@ def add_benchmark_args(parser: argparse.ArgumentParser): | |||
| action=argparse.BooleanOptionalAction, | |||
| help="drop tables before starting", | |||
| ) | |||
| parser.add_argument( | |||
There was a problem hiding this comment.
Can we achieve the desired auto-scaling by just running queries? We already have a mechanism for that, we just need to expose it in the cli arguments
There was a problem hiding this comment.
Ideally we don't want to create a bunch of tables just to warm up the cluster
There was a problem hiding this comment.
Change to scale up queries only, expose the warmup queries parameter which default to num of queries and can be specified by users
|
@QianZhu Please see if the new warmup_queries mechanism is what you are looking for? |
No description provided.