xiangfu0 commented on PR #19568: URL: https://github.com/apache/pinot/pull/19568#issuecomment-5803777768
**CI note: `Pinot Integration Test Set 2` failed on `b08930a5a7`, and the failure is not from this PR.** It failed in *Build Project* after ~2 minutes, before any test ran: ``` Failed to execute goal org.apache.rat:apache-rat-plugin:0.18:check (default) on project pinot-timeseries-lang: Execution default of goal org.apache.rat:apache-rat-plugin:0.18:check failed. ConcurrentModificationException ``` Why it isn't this PR's: - `pinot-timeseries-lang` is not touched by this PR. - RAT went from 0.16.1 to **0.18** on master a few hours ago via #19633 (Apache parent 39 → 40, which pins `version.apache-rat-plugin`); the rebase onto `33d88e480b` picked it up. - The same *Pinot Integration Tests* workflow **passed on master at this PR's exact base** `33d88e480b` with RAT 0.18, so it is not a deterministic break. - The build step runs `mvn ... -T 16`. A `ConcurrentModificationException` in a license checker that shows up only sometimes points to plugin state shared across parallel module builds. No fix exists yet. A candidate patch, which I'd rather land as its own PR than widen this one with: pin RAT back in the root `pom.xml` `<properties>` until 0.18's parallel-build behaviour is sorted out: ```xml <version.apache-rat-plugin>0.16.1</version.apache-rat-plugin> ``` I'll re-run the failed job once, when the current run completes. -- 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]
