Open
Description
New Feature / Enhancement Checklist
- I am not disclosing a vulnerability.I am not just asking a question.I have searched through existing issues.
Current Limitation
Parse Server has no CI check to compare performance impact of a PR before merging.
Feature / Enhancement Description
Add performance impact CI check to validate a positive / prevent negative performance impact of a change.
- The perf impact should be displayed inside the PR thread, either as a check status message with link to a more detailed page, or an auto-updated comment.
- There are some internal tools available such as the Parse GitHub Assistant that could run a performance script on a branch and update a CI check accordingly.
- If you want to take on this issue as part of the Bounty Program, it is recommended to begin a discussion about how to design and implement this.
Example Use Case
n/a
Alternatives / Workarounds
n/a
Metadata
Metadata
Assignees
Type
Projects
Milestone
Relationships
Development
No branches or pull requests
Activity
parse-github-assistant commentedon Oct 9, 2021
Thanks for opening this issue!
mtrezza commentedon Oct 14, 2021
Possibly this:
https://github.com/marketplace/actions/continuous-benchmark
benchmark-action/github-action-benchmark#68
We can simply use the perf measure API of Node 16 and the tool above can read and process it.
@dplewis Could that replace our internal benchmark tool?
dblythy commentedon Mar 8, 2023
Possibly a duplicate of #7610
mtrezza commentedon Mar 8, 2023
This is #7610 🙃
dblythy commentedon Mar 8, 2023
Oops
encode.js
parse-community/Parse-SDK-JS#2099