-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
GH-36218: [CI][Go] Run benchmark steps only on the main branch #36219
Conversation
Because our benchmark tool supports only the main branch.
|
Oh, sorry. You're right. Fixed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
|
Sorry. This is a follow-up of apacheGH-36219. We need one more change for this.
### Rationale for this change Sorry. This is a follow-up of GH-36219. We need one more change for this. ### What changes are included in this PR? Add one more branch name check. ### Are these changes tested? No. ### Are there any user-facing changes? No. * Closes: #36218 Authored-by: Sutou Kouhei <kou@clear-code.com> Signed-off-by: Sutou Kouhei <kou@clear-code.com>
Conbench analyzed the 6 benchmark runs on commit There were 3 benchmark results indicating a performance regression:
The full Conbench report has more details. |
Rationale for this change
Because our benchmark tool supports only the main branch.
What changes are included in this PR?
Add branch name check to the existing
if:
s.Are these changes tested?
No.
Are there any user-facing changes?
No.