-
Notifications
You must be signed in to change notification settings - Fork 4.1k
[Java] Cleanup protobuf-maven-plugin usage #40843
Copy link
Copy link
Closed
Description
Describe the enhancement requested
Arrow uses protobuf-maven-plugin in several modules but not in the recommended way. There are multiple issues like:
- use of
compile/compile-custominstead oftest-compile/test-compile-customfor test protobufs (causing test classes to be added to the main artifact) - forcing the output location which causes different execution to overwrite each other (each execution should have their own output directory). When done properly, setting
clearOutputDirectoryproperty tofalseis not necessary anymore - adding
test-compilegoal when no test protobufs are present.
The protobuf-maven-plugin is also declared multiple times instead of it managed at the top level
Component(s)
Java
Reactions are currently unavailable