Skip to content

Build & Publish Thrift Maven Plugin#3347

Open
jmkeyes wants to merge 2 commits intoapache:masterfrom
jmkeyes:thrift-maven-plugin-build
Open

Build & Publish Thrift Maven Plugin#3347
jmkeyes wants to merge 2 commits intoapache:masterfrom
jmkeyes:thrift-maven-plugin-build

Conversation

@jmkeyes
Copy link

@jmkeyes jmkeyes commented Mar 17, 2026

This change builds the thrift-maven-plugin in the contrib folder with Maven by default.

The intention is to keep org.apache.thrift:thrift-maven-plugin updated as Thrift evolves.

@Jens-G
Copy link
Member

Jens-G commented Mar 21, 2026

  1. What means "keep updated as Thrift evolves"?

  2. Would the patch be compatible with what the ASF ReleasePolicy has to say about what can and what cannot be published?

@Jens-G Jens-G added the java Pull requests that update Java code label Mar 21, 2026
@jmkeyes
Copy link
Author

jmkeyes commented Mar 21, 2026

  1. What means "keep updated as Thrift evolves"?

I would like for the thrift-maven-plugin to be published along with new versions of Thrift so I can generate Java bindings for my applications with a native Maven plugin.

  1. Would the patch be compatible with what the ASF ReleasePolicy has to say about what can and what cannot be published?

I believe so but I don't know for sure. I saw that the Thrift Maven plugin was available within the Thrift project's source code so I created a patch to build it in a way similar to other Java parts of Thrift.

If the Maven plugin in Thrift's source code is not compliant with the Apache Foundation's release policy, then I'd like to know what I could change to make it acceptable, as the Maven plugin provides a lot of value.

@Jens-G
Copy link
Member

Jens-G commented Mar 21, 2026

I just wonder because we usually publish the Java packages via maven. Als I'm being told the maven plugin did not change for a long time. Disclaimer: I'm no Java user myself, so pls bear with me if I misrepresent sth and/or my knowledge is outdated. I might add that the Java publishing is always kind of a hassle, so if we can make it better, I'm open to it.

@jeking3 thoughts?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

java Pull requests that update Java code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants