This is an automated email from the ASF dual-hosted git repository. ppkarwasz pushed a commit to branch trunk in repository https://gitbox.apache.org/repos/asf/logging-flume.git
commit f401fa9a7c66e84e2f3a65af5a8e77712709c42f Author: Piotr P. Karwasz <[email protected]> AuthorDate: Sat Aug 1 00:49:04 2026 +0200 Adjust PMD violation ceilings to PMD 7 counts (#485) The maven-pmd-plugin 3.15.0 to 3.28.0 bump (#479) switched from PMD 6 to PMD 7.17.0, which finds a different violation mix in most modules. Ratchet each module's `pmd.maxAllowedViolations` to the new actual count so the build passes again. The counts rise in some modules and drop in others (flume-ng-sdk goes from 170 to 30); fixing the violations themselves remains a TODO. Assisted-By: Claude Fable 5 <[email protected]> --- flume-ng-auth/pom.xml | 2 +- flume-ng-channels/flume-file-channel/pom.xml | 2 +- flume-ng-channels/flume-spillable-memory-channel/pom.xml | 2 +- flume-ng-configuration/pom.xml | 2 +- flume-ng-core/pom.xml | 2 +- flume-ng-instrumentation/flume-ganglia-monitor/pom.xml | 3 ++- flume-ng-node/pom.xml | 2 +- flume-ng-sdk/pom.xml | 2 +- flume-ng-sources/flume-syslog-source/pom.xml | 3 ++- flume-ng-sources/flume-taildir-source/pom.xml | 2 +- 10 files changed, 12 insertions(+), 10 deletions(-) diff --git a/flume-ng-auth/pom.xml b/flume-ng-auth/pom.xml index ae6c7dbb..e288313d 100644 --- a/flume-ng-auth/pom.xml +++ b/flume-ng-auth/pom.xml @@ -32,7 +32,7 @@ <properties> <!-- TODO fix spotbugs/pmd violations --> <spotbugs.maxAllowedViolations>6</spotbugs.maxAllowedViolations> - <pmd.maxAllowedViolations>7</pmd.maxAllowedViolations> + <pmd.maxAllowedViolations>5</pmd.maxAllowedViolations> <module.name>org.apache.flume.auth</module.name> </properties> diff --git a/flume-ng-channels/flume-file-channel/pom.xml b/flume-ng-channels/flume-file-channel/pom.xml index 70c26cd0..245e4d56 100644 --- a/flume-ng-channels/flume-file-channel/pom.xml +++ b/flume-ng-channels/flume-file-channel/pom.xml @@ -31,7 +31,7 @@ <properties> <!-- TODO fix spotbugs violations --> <spotbugs.maxAllowedViolations>162</spotbugs.maxAllowedViolations> - <pmd.maxAllowedViolations>19</pmd.maxAllowedViolations> + <pmd.maxAllowedViolations>21</pmd.maxAllowedViolations> <module.name>org.apache.flume.channel.file</module.name> </properties> diff --git a/flume-ng-channels/flume-spillable-memory-channel/pom.xml b/flume-ng-channels/flume-spillable-memory-channel/pom.xml index 70c8d541..b891a42a 100644 --- a/flume-ng-channels/flume-spillable-memory-channel/pom.xml +++ b/flume-ng-channels/flume-spillable-memory-channel/pom.xml @@ -31,7 +31,7 @@ <properties> <!-- TODO fix spotbugs/pmd violations --> <spotbugs.maxAllowedViolations>11</spotbugs.maxAllowedViolations> - <pmd.maxAllowedViolations>8</pmd.maxAllowedViolations> + <pmd.maxAllowedViolations>4</pmd.maxAllowedViolations> <module.name>org.apache.flume.channel.spillable.memory</module.name> </properties> diff --git a/flume-ng-configuration/pom.xml b/flume-ng-configuration/pom.xml index 1c86eaf9..8ab25fd6 100644 --- a/flume-ng-configuration/pom.xml +++ b/flume-ng-configuration/pom.xml @@ -29,7 +29,7 @@ <properties> <!-- TODO fix spotbugs violations --> <spotbugs.maxAllowedViolations>36</spotbugs.maxAllowedViolations> - <pmd.maxAllowedViolations>17</pmd.maxAllowedViolations> + <pmd.maxAllowedViolations>13</pmd.maxAllowedViolations> <module.name>org.apache.flume.conf</module.name> </properties> diff --git a/flume-ng-core/pom.xml b/flume-ng-core/pom.xml index 61c505c9..e8b6a0d2 100644 --- a/flume-ng-core/pom.xml +++ b/flume-ng-core/pom.xml @@ -33,7 +33,7 @@ <properties> <!-- TODO fix spotbugs violations --> <spotbugs.maxAllowedViolations>152</spotbugs.maxAllowedViolations> - <pmd.maxAllowedViolations>64</pmd.maxAllowedViolations> + <pmd.maxAllowedViolations>69</pmd.maxAllowedViolations> <module.name>org.apache.flume.core</module.name> </properties> diff --git a/flume-ng-instrumentation/flume-ganglia-monitor/pom.xml b/flume-ng-instrumentation/flume-ganglia-monitor/pom.xml index 2e531ee8..1911b120 100644 --- a/flume-ng-instrumentation/flume-ganglia-monitor/pom.xml +++ b/flume-ng-instrumentation/flume-ganglia-monitor/pom.xml @@ -32,8 +32,9 @@ <properties> <!-- This artifact was never released before --> <bnd.baseline.fail.on.missing>false</bnd.baseline.fail.on.missing> - <!-- TODO fix spotbugs violations --> + <!-- TODO fix spotbugs/pmd violations --> <spotbugs.maxAllowedViolations>13</spotbugs.maxAllowedViolations> + <pmd.maxAllowedViolations>3</pmd.maxAllowedViolations> <module.name>org.apache.flume.instrumentation.ganglia</module.name> </properties> diff --git a/flume-ng-node/pom.xml b/flume-ng-node/pom.xml index 4fae3f83..40f6a4d0 100644 --- a/flume-ng-node/pom.xml +++ b/flume-ng-node/pom.xml @@ -32,7 +32,7 @@ <properties> <!-- TODO fix spotbugs violations --> <spotbugs.maxAllowedViolations>12</spotbugs.maxAllowedViolations> - <pmd.maxAllowedViolations>7</pmd.maxAllowedViolations> + <pmd.maxAllowedViolations>5</pmd.maxAllowedViolations> <module.name>org.apache.flume.node</module.name> <junit-jupiter.version>6.1.2</junit-jupiter.version> </properties> diff --git a/flume-ng-sdk/pom.xml b/flume-ng-sdk/pom.xml index a5c0a4a0..bbe85ee1 100644 --- a/flume-ng-sdk/pom.xml +++ b/flume-ng-sdk/pom.xml @@ -32,7 +32,7 @@ <properties> <!-- TODO fix spotbugs/pmd violations --> <spotbugs.maxAllowedViolations>69</spotbugs.maxAllowedViolations> - <pmd.maxAllowedViolations>170</pmd.maxAllowedViolations> + <pmd.maxAllowedViolations>30</pmd.maxAllowedViolations> <module.name>org.apache.flume.sdk</module.name> </properties> diff --git a/flume-ng-sources/flume-syslog-source/pom.xml b/flume-ng-sources/flume-syslog-source/pom.xml index d169d164..e45b597f 100644 --- a/flume-ng-sources/flume-syslog-source/pom.xml +++ b/flume-ng-sources/flume-syslog-source/pom.xml @@ -32,8 +32,9 @@ <properties> <!-- This artifact was never released before --> <bnd.baseline.fail.on.missing>false</bnd.baseline.fail.on.missing> - <!-- TODO fix spotbugs violations --> + <!-- TODO fix spotbugs/pmd violations --> <spotbugs.maxAllowedViolations>8</spotbugs.maxAllowedViolations> + <pmd.maxAllowedViolations>1</pmd.maxAllowedViolations> <module.name>org.apache.flume.source.syslog</module.name> </properties> diff --git a/flume-ng-sources/flume-taildir-source/pom.xml b/flume-ng-sources/flume-taildir-source/pom.xml index c4e34bb0..89edc395 100644 --- a/flume-ng-sources/flume-taildir-source/pom.xml +++ b/flume-ng-sources/flume-taildir-source/pom.xml @@ -32,7 +32,7 @@ <properties> <!-- TODO fix spotbugs/pmd violations --> <spotbugs.maxAllowedViolations>24</spotbugs.maxAllowedViolations> - <pmd.maxAllowedViolations>3</pmd.maxAllowedViolations> + <pmd.maxAllowedViolations>4</pmd.maxAllowedViolations> <module.name>org.apache.flume.source.taildir</module.name> </properties>
