Skip to content

chore: Run benchmarks in CI#425

Merged
kodiakhq[bot] merged 19 commits intomainfrom
benchmark-ci
Nov 24, 2022
Merged

chore: Run benchmarks in CI#425
kodiakhq[bot] merged 19 commits intomainfrom
benchmark-ci

Conversation

@hermanschaaf
Copy link
Copy Markdown
Contributor

@hermanschaaf hermanschaaf commented Nov 22, 2022

Adds a Github Action step to run benchmarks in CI and report on the differences with main.

It uses a combination of gobenchdata, the netlify/delta-action and a simple Go script to glue them together.

Example from lower down in this PR:
Screenshot 2022-11-22 at 18 56 21

Closes #424

@github-actions
Copy link
Copy Markdown

github-actions bot commented Nov 22, 2022

⏱️ Benchmark results

  • DefaultConcurrency-2 resources/s: 12,358
  • Glob-2 ns/op: 146.9
  • TablesWithChildrenDefaultConcurrency-2 resources/s: 31,478
  • BufferedScanner-2 ns/op: 9.28
  • LogReader-2 ns/op: 30.56

if: matrix.os == 'ubuntu-latest'
run: make benchmark-ci
- name: Delta
uses: hermanschaaf/delta-action@v1.0.7
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

We can change this to the original upstream version if/when my PR gets merged: netlify/delta-action#331

@hermanschaaf hermanschaaf marked this pull request as ready for review November 22, 2022 18:56
@kodiakhq kodiakhq bot merged commit 9882458 into main Nov 24, 2022
@kodiakhq kodiakhq bot deleted the benchmark-ci branch November 24, 2022 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Run benchmarks in CI

2 participants