-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
MINOR: [Java] Remove superfluous add-source execution #43216
MINOR: [Java] Remove superfluous add-source execution #43216
Conversation
protobuf plugin already adds generated protobuf classes to the list of source directories, so remove the superfluous build-helper:add-source execution
@github-actions crossbow submit -g java |
Revision: f223537 Submitted crossbow builds: ursacomputing/crossbow @ actions-d29bec0db0 |
After merging your PR, Conbench analyzed the 4 benchmarking runs that have been run so far on merge-commit 797ca30. There were no benchmark performance regressions. 🎉 The full Conbench report has more details. |
@github-actions crossbow submit -g java-jars |
|
@github-actions crossbow submit -g java java-jars |
Revision: f223537 Submitted crossbow builds: ursacomputing/crossbow @ actions-ba68e21443 |
Rationale for this change
protobuf plugin already adds generated protobuf classes to the list of source directories, so remove the superfluous build-helper:add-source execution
Are these changes tested?
Yes via CI/CD
Are there any user-facing changes?
No