rfscholte commented on code in PR #14015:
URL: https://github.com/apache/pinot/pull/14015#discussion_r1766442395
##########
pinot-common/pom.xml:
##########
@@ -100,22 +100,31 @@
<!-- "Plugs in" the Calcite's Parser.jj with the variables present in
config.fmpp. These contain the custom rules
as well as the class to which the custom implementation will get
generated -->
<plugin>
- <groupId>com.googlecode.fmpp-maven-plugin</groupId>
- <artifactId>fmpp-maven-plugin</artifactId>
+ <groupId>org.apache.maven.plugins</groupId>
+ <artifactId>maven-antrun-plugin</artifactId>
<executions>
<execution>
- <id>generate-fmpp-sources</id>
<phase>generate-sources</phase>
- <goals>
- <goal>generate</goal>
- </goals>
<configuration>
- <cfgFile>${project.build.directory}/codegen/config.fmpp</cfgFile>
-
<outputDirectory>${project.build.directory}/generated-sources</outputDirectory>
-
<templateDirectory>${project.build.directory}/codegen/templates</templateDirectory>
+ <target name="generate-code">
+ <taskdef name="fmpp" classname="fmpp.tools.AntTask"
classpathref="maven.compile.classpath"/>
+ <fmpp
configuration="${project.build.directory}/codegen/config.fmpp"
+ sourceRoot="${project.build.directory}/codegen/templates"
+ outputRoot="${project.build.directory}/generated-sources"/>
+ </target>
</configuration>
+ <goals>
+ <goal>run</goal>
+ </goals>
</execution>
</executions>
+ <dependencies>
+ <dependency>
+ <groupId>net.sourceforge.fmpp</groupId>
+ <artifactId>fmpp</artifactId>
+ <version>${fmpp.version}</version>
Review Comment:
It is only used in this place, but looking at the pinot-project where every
version is managed in the root pom, that would make sense. (personally I would
also put them in alphabetic order, but that's something for another PR)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
