search

memsql / dbbench

Database Benchmark Tool
Apache License 2.0
155 stars 30 forks source link

Add command line option to override default connect param #5

Closed robert-s-lee closed 7 years ago

awreece commented 7 years ago

This looks pretty good, only one thing: if this is expected to be set via a command line flag, we should probably add a corresponding call to flag.StringVar to dbbench.go:init().

Also, we might also want to make SQL server respect the param configuration as well.