Skip to content
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

Performace tests investigation #143

Open
mantzas opened this issue Jan 17, 2020 · 4 comments
Open

Performace tests investigation #143

mantzas opened this issue Jan 17, 2020 · 4 comments
Labels
Milestone

Comments

@mantzas
Copy link

mantzas commented Jan 17, 2020

Is your feature request related to a problem? Please describe

There is a need to have performance tests for Patron.
Let's investigate how to do this.

@tpaschalis
Copy link

I suppose one of the most basic ideas is to strategically place some benchmarks, and compare them between commits, right? There are some built-in tools like benchstat and benchcmp which we can use. We could also :

a) have a Makefile command that compares HEAD vs origin/master, and fails if the regression is over a threshold
b) build a Github Actions workflow that does the same thing, and open-source it under the beatlabs organization
c) use some off-the-shelf solution like gobenchdata, gobenchui or cob to keep track of performance over time

Some thoughts :

  • It shouldn't be a hard requirement; complexity might increase due to adding features and not due to accidental performance regression
  • These benchmarks should be relatively stable, so that don't end up comparing apples and oranges

@mantzas
Copy link
Author

mantzas commented Jul 6, 2020

There is only the problem with infra. Everything runs on VM's so there is no guaratee on perfromance... tricky business

@tpaschalis
Copy link

tpaschalis commented Jul 6, 2020

Hmm, you're right. The benchstat tool actually reports differences only if they're statistically significant which should somewhat normalize this fluctuation. To do so it requires 4-5 runs per benchmark before and after to provide an answer which makes the benchmarks slower though. I'll keep looking into this

@mantzas
Copy link
Author

mantzas commented Jul 6, 2020

I do not have a good answer on how to proceed with this. Maybe it is good enough since GitHub action has a specific instance type available for the CI. If they change the instance type to something else we will see an improvement/impact.

@tpaschalis tpaschalis added this to the Unplanned milestone Oct 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants