This is an automated email from the ASF dual-hosted git repository.

bipinprasad pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/storm.git


The following commit(s) were added to refs/heads/master by this push:
     new 67cb564e6 [STORM-3853] Change version on maven-pmd-plugin to 3.16.0 
and fix errors for "mvn pmd:check" (#3471)
67cb564e6 is described below

commit 67cb564e6afb836a08a7d91c1ae0455e16f1952f
Author: Bipin Prasad <[email protected]>
AuthorDate: Tue Apr 26 08:21:57 2022 -0700

    [STORM-3853] Change version on maven-pmd-plugin to 3.16.0 and fix errors 
for "mvn pmd:check" (#3471)
---
 .../src/jvm/org/apache/storm/starter/AnchoredWordCount.java            | 3 ++-
 .../src/jvm/org/apache/storm/starter/ExclamationTopology.java          | 1 +
 .../storm-starter/src/jvm/org/apache/storm/starter/LambdaTopology.java | 1 -
 .../java/org/apache/storm/container/oci/HdfsOciResourcesLocalizer.java | 2 ++
 pom.xml                                                                | 2 +-
 5 files changed, 6 insertions(+), 3 deletions(-)

diff --git 
a/examples/storm-starter/src/jvm/org/apache/storm/starter/AnchoredWordCount.java
 
b/examples/storm-starter/src/jvm/org/apache/storm/starter/AnchoredWordCount.java
index 022f6ad2a..1809cf19f 100644
--- 
a/examples/storm-starter/src/jvm/org/apache/storm/starter/AnchoredWordCount.java
+++ 
b/examples/storm-starter/src/jvm/org/apache/storm/starter/AnchoredWordCount.java
@@ -1,4 +1,4 @@
-/**
+/*
  * Licensed to the Apache Software Foundation (ASF) under one or more 
contributor license agreements.  See the NOTICE file distributed with
  * this work for additional information regarding copyright ownership.  The 
ASF licenses this file to you under the Apache License, Version
  * 2.0 (the "License"); you may not use this file except in compliance with 
the License.  You may obtain a copy of the License at
@@ -32,6 +32,7 @@ import org.apache.storm.utils.Utils;
 
 public class AnchoredWordCount extends ConfigurableTopology {
 
+    @Override
     protected int run(String[] args) throws Exception {
         TopologyBuilder builder = new TopologyBuilder();
 
diff --git 
a/examples/storm-starter/src/jvm/org/apache/storm/starter/ExclamationTopology.java
 
b/examples/storm-starter/src/jvm/org/apache/storm/starter/ExclamationTopology.java
index 22cb2ba83..31854f613 100644
--- 
a/examples/storm-starter/src/jvm/org/apache/storm/starter/ExclamationTopology.java
+++ 
b/examples/storm-starter/src/jvm/org/apache/storm/starter/ExclamationTopology.java
@@ -33,6 +33,7 @@ public class ExclamationTopology extends ConfigurableTopology 
{
         ConfigurableTopology.start(new ExclamationTopology(), args);
     }
 
+    @Override
     protected int run(String[] args) {
         TopologyBuilder builder = new TopologyBuilder();
 
diff --git 
a/examples/storm-starter/src/jvm/org/apache/storm/starter/LambdaTopology.java 
b/examples/storm-starter/src/jvm/org/apache/storm/starter/LambdaTopology.java
index e7c134b58..d5dec2968 100644
--- 
a/examples/storm-starter/src/jvm/org/apache/storm/starter/LambdaTopology.java
+++ 
b/examples/storm-starter/src/jvm/org/apache/storm/starter/LambdaTopology.java
@@ -12,7 +12,6 @@
 
 package org.apache.storm.starter;
 
-import java.io.Serializable;
 import java.util.UUID;
 import org.apache.storm.Config;
 import org.apache.storm.topology.ConfigurableTopology;
diff --git 
a/external/storm-hdfs-oci/src/main/java/org/apache/storm/container/oci/HdfsOciResourcesLocalizer.java
 
b/external/storm-hdfs-oci/src/main/java/org/apache/storm/container/oci/HdfsOciResourcesLocalizer.java
index 56ffa9590..ef749c25f 100644
--- 
a/external/storm-hdfs-oci/src/main/java/org/apache/storm/container/oci/HdfsOciResourcesLocalizer.java
+++ 
b/external/storm-hdfs-oci/src/main/java/org/apache/storm/container/oci/HdfsOciResourcesLocalizer.java
@@ -47,6 +47,7 @@ public class HdfsOciResourcesLocalizer implements 
OciResourcesLocalizerInterface
      * @param conf the storm conf.
      * @throws IOException on I/O exception
      */
+    @Override
     public void init(Map<String, Object> conf) throws IOException {
         //login to hdfs
         HadoopLoginUtil.loginHadoop(conf);
@@ -66,6 +67,7 @@ public class HdfsOciResourcesLocalizer implements 
OciResourcesLocalizerInterface
      * @return the destination of the oci resource
      * @throws IOException on I/O exception
      */
+    @Override
     public synchronized String localize(OciResource ociResource) throws 
IOException {
         if (ociResource == null) {
             return null;
diff --git a/pom.xml b/pom.xml
index ffb4fa503..84f83f4c5 100644
--- a/pom.xml
+++ b/pom.xml
@@ -1263,7 +1263,7 @@
                 <plugin>
                     <groupId>org.apache.maven.plugins</groupId>
                     <artifactId>maven-pmd-plugin</artifactId>
-                    <version>3.12.0</version>
+                    <version>3.16.0</version>
                     <configuration>
                         <rulesets>
                             <ruleset>storm/pmd-ruleset.xml</ruleset>

Reply via email to