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.

Sign up for GitHub

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

Jump to bottom

Temporarily remove throughput benchmark #555

Merged

afsalthaj merged 1 commit into main from throughput

Jun 7, 2024

Contributor

afsalthaj commented Jun 7, 2024

I have to remove this to see a complete workflow of benchmark

Throughput benchmark kept failing in my machine. Firstly we need to ensure its running as consistent as other benchmarks before having this as part of the CI workflow.
I am currently looking at this.
This PR is to unblock rest of the benchmark jobs
Also as I mentioned earlier, it was a good idea to keep the number of jobs minimal when we test a workflows, and that's why I didn't have all of these jobs and it wasn't accidental


          Temporarily remove throughput

bdeb2b7

afsalthaj changed the title ~~Temporarily remove throughput~~ Temporarily remove throughput benchmark

Contributor Author

afsalthaj commented Jun 7, 2024

I am merging this as it is a straight forward change, not impacting any other part of the code.

afsalthaj merged commit ba6265c into main

13 checks passed

afsalthaj deleted the throughput branch

June 7, 2024 05:54

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet