Skip to content

KAFKA-21064 Apache RAT Gradle plugin version upgrade; redundant custom style file is removed - #23424

Draft
dejan2609 wants to merge 4 commits into
apache:trunkfrom
dejan2609:KAFKA-21064
Draft

dejan2609 wants to merge 4 commits into
apache:trunkfrom
dejan2609:KAFKA-21064

Conversation

@dejan2609

@dejan2609 dejan2609 commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot added triage PRs from the community build Gradle build or GitHub Actions labels Sep 10, 2026
@dejan2609

dejan2609 commented Sep 10, 2026

Copy link
Copy Markdown
Contributor Author

Rationale for custom style file gradle/resources/rat-output-to-html.xsl removal: this style is equivalent to a default style.

Steps to verify this claim (use Kafka trunk branch):

  • comment out file reference: build.gradle:stylesheet.set(file('gradle/resources/rat-output-to-html.xsl'))
  • execute: ./gradlew rat
  • compare two styles: diff build/tmp/rat/default-stylesheet.xsl gradle/resources/rat-output-to-html.xsl
69a70,71
>   <p>This HTML version (yes, it is!) is generated from the RAT xml reports using Saxon9B. All the outputs required are displayed below, similar to the .txt version.
>            This is obviously a work in progress; and a prettier, easier to read and manage version will be available soon</p>

@dejan2609

Copy link
Copy Markdown
Contributor Author

Old report vs. new report:
image

image

@dejan2609

dejan2609 commented Sep 11, 2026

Copy link
Copy Markdown
Contributor Author

Update: plugin version 0.10.0 takes additional time to build:

Kafka trunk (plugin version 0.8.1): ~ 40 seconds
PR mentioned above (plugin version 0.10.0): ~ 1minute and 40 seconds

so I created plugin-related issue (see above).

Plugin developer was extra responsive: root cause for this additional time is Apache RAT upgrade (0.15 --> 0.17) so he was kind enough to release a new plugin version 0.11.0 that's using Apache RAT 0.18.

All-in-all: I'll bump plugin version to 0.11.0

@dejan2609 dejan2609 changed the title KAFKA-21064 Apache RAT Gradle plugin version upgrade: 0.8.1 --> 0.10.0; redundant custom style file is removed KAFKA-21064 Apache RAT Gradle plugin version upgrade; redundant custom style file is removed Sep 11, 2026
@dejan2609

dejan2609 commented Sep 11, 2026

Copy link
Copy Markdown
Contributor Author

Update: ./gradlew rat --rerun-tasks --no-daemon task execution duration (with recent plugin versions) on my laptop:

  • Kafka trunk, plugin version: 0.8.1 (RAT: 0.15) --> ~ 42s
  • plugin version: 0.10.0 (RAT: 0.17) --> ~ 2m 5s
  • plugin version: 0.11.0 (RAT: 0.18) --> ~ 1m 20s

@dejan2609

Copy link
Copy Markdown
Contributor Author

Demoting to a draft (additional configuration is required).

@dejan2609
dejan2609 marked this pull request as draft September 14, 2026 15:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build Gradle build or GitHub Actions triage PRs from the community

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant