WIP: AVRO-4223 Gradle plugin for generating Java code

[frevib]

What is the purpose of the change

Gradle plugin to generate Java code from Avro files

Verifying this change

This change added tests and can be verified as follows:

cd to avro/lang/java/gradle-plugin

./gradlew test

Documentation

• Does this pull request introduce a new feature? (yes)
• If yes, how is the feature documented? (docs): https://github.com/frevib/avro/blob/AVRO-4223-gradle-plugin/lang/java/gradle-plugin/README.md

Release

0.0.2 is released and fully works with AVSC files: https://central.sonatype.com/artifact/eu.eventloopsoftware.avro-gradle-plugin/eu.eventloopsoftware.avro-gradle-plugin.gradle.plugin/versions - read docs (above) how to use

0.0.5 https://plugins.gradle.org/plugin/eu.eventloopsoftware.avro-gradle-plugin

0.0.8 https://plugins.gradle.org/plugin/eu.eventloopsoftware.avro-gradle-plugin

0.1.0 https://plugins.gradle.org/plugin/eu.eventloopsoftware.avro-gradle-plugin - this release adds Protocol support.

Protocol support has been added in this release

Installation instructions: https://github.com/frevib/avro/blob/AVRO-4223-gradle-plugin/lang/java/gradle-plugin/README.md#version

An official release will be done in the coming month

[frevib]

Avro gradle plugin 0.1.0 is released: https://plugins.gradle.org/plugin/eu.eventloopsoftware.avro-gradle-plugin

Protocol support has been added in this release

[frevib]

@raphaelauv did you test the latest release https://plugins.gradle.org/plugin/eu.eventloopsoftware.avro-gradle-plugin? No need to add compileJava.source(avroGenerateJavaClasses) any more.

[Sineaggi]

Instead of this, we probably want to add some pre-configured configuration, called avroSources or something that way people can add their dependencies as normal, like

dependencies {
    avroSources("eu.eventloopsoftware.group-id:artifact-id:1.0.0") // ...

Certain plugins ship their own dependencies block (like in Gradle's test suites plugin) so perhaps something like

avro {
    sourceZipFiles {
        dependencies {
            avroSources("eu.eventloopsoftware.group-id:artifact-id:1.0.0") // ...

could be used.

[opwvhk]

Is is possible to add the compile- & runtime dependencies while generating code? Then we wouldn't need any configuration at all.

[Sineaggi]

@opwvhk I'm not quite following? I think the dependencies here was just to allow people to add avro sources that are zipped in jars deployed to a standard maven repo.

[Sineaggi]

Otherwise, if we at least made sourceZipFiles a ConfigurableFileCollection, then users could configure either via files, or a their own custom configuration.

[frevib]

@Sineaggi you're right, this is to allow ppl to add Avro sources that are in JAR files. Maybe the example with the full path isn't the best one. I've added an example with Configuration. What do think now @Sineaggi ?

[opwvhk]

This is yet another marked improvement. Thank you!

I have a few open comments, and we'll want to fix the build, but otherwise I'm happy with the result.

[opwvhk]

Can we pass the Maven log level here? I doubt it, but it would be convenient.

[frevib]

Which Maven log level? The Maven log level can be passed when this Maven command is run, which in turn runs this Gradle command: ./gradlew assemble -i. Passing a Maven argument here would be on the wrong level, one too deep.

[martin-g]

I have created https://issues.apache.org/jira/browse/INFRA-27616 for the requirement from Gradle to prove the ownership of avro.apache.org DNS domain.