[FLINK-35999] Migrate from fmpp-maven-plugin to drill-fmpp-maven-plugin

[r-sidd]

What is the purpose of the change

This PR will remove the log4j dependency from the fmpp-maven-plugin

Brief change log

In the maven package - flink-sql-parser, the fmpp-maven-plugin has a dependency of xbean-reflect, which in-turn has a dependency of log4j jar 1.12. We can exclude this dependency from xbean-reflect. Attaching details below:

https://mvnrepository.com/artifact/org.apache.xbean/xbean-reflect/3.4

https://github.com/apache/flink/blob/master/flink-table/flink-sql-parser/pom.xml

Documentation

• Does this pull request introduce a new feature? NO

[flinkbot]

CI report:

• 9ff997aad5e0beceaaf78e65ad60eba80e9f1a1e Azure: SUCCESS

Bot commands

The @flinkbot bot supports the following commands:

• @flinkbot run azure re-run the last Azure build

[snuyanzin]

Thanks for the contribution

I would suggest rather move to drill-fmpp-maven-plugin https://github.com/apache/drill/blob/master/tools/fmpp/pom.xml which is based fmpp-maven-plugin and already excludes some dependencies including log4j

Also Calcite depended on this plugin as well (before it was migrated to gradle)

[r-sidd]

Thanks for the contribution

I would suggest rather move to drill-fmpp-maven-plugin https://github.com/apache/drill/blob/master/tools/fmpp/pom.xml which is based fmpp-maven-plugin and already excludes some dependencies including log4j

Also Calcite depended on this plugin as well (before it was migrated to gradle)

Okay sure, makes sense, will migrate to drill-fmpp-maven-plugin :)

[r-sidd]

@flinkbot run azure

[r-sidd]

Hey @snuyanzin thanks for the suggestion, I've made the necessary changes and ran the tests in local, please review the same :)