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

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


The following commit(s) were added to refs/heads/master by this push:
     new 3dd5044a2 OOZIE-3694 [sharelib-hive] Fix current SpotBugs discovered 
issues in Oozie's sharelib-hive module (jmakai via dionusos)
3dd5044a2 is described below

commit 3dd5044a2835cf21f0aa4d3626cacbfaea5886c8
Author: Denes Bodo <[email protected]>
AuthorDate: Fri Feb 3 09:37:21 2023 +0100

    OOZIE-3694 [sharelib-hive] Fix current SpotBugs discovered issues in 
Oozie's sharelib-hive module (jmakai via dionusos)
---
 release-log.txt                                                      | 1 +
 sharelib/hive/pom.xml                                                | 5 +++++
 .../hive/src/main/java/org/apache/oozie/action/hadoop/HiveMain.java  | 3 +++
 3 files changed, 9 insertions(+)

diff --git a/release-log.txt b/release-log.txt
index 3de89b797..9eda2c7f5 100644
--- a/release-log.txt
+++ b/release-log.txt
@@ -1,5 +1,6 @@
 -- Oozie 5.3.0 release (trunk - unreleased)
 
+OOZIE-3694 [sharelib-hive] Fix current SpotBugs discovered issues in Oozie's 
sharelib-hive module (jmakai via dionusos)
 OOZIE-3690 [server] Fix current SpotBugs discovered issues in Oozie's server 
module (jmakai via dionusos)
 OOZIE-3692 [sharelib-spark] Fix current SpotBugs discovered issues in Oozie's 
sharelib-spark module (jmakai via dionusos)
 OOZIE-3693 [examples] Fix current SpotBugs discovered issues in Oozie's 
examples module (jmakai via dionusos)
diff --git a/sharelib/hive/pom.xml b/sharelib/hive/pom.xml
index dfc042cef..58e87d0c7 100644
--- a/sharelib/hive/pom.xml
+++ b/sharelib/hive/pom.xml
@@ -196,6 +196,11 @@
             <artifactId>jline</artifactId>
             <version>${hive.jline.version}</version>
         </dependency>
+        <dependency>
+            <groupId>com.github.spotbugs</groupId>
+            <artifactId>spotbugs-annotations</artifactId>
+            <scope>provided</scope>
+        </dependency>
     </dependencies>
 
     <build>
diff --git 
a/sharelib/hive/src/main/java/org/apache/oozie/action/hadoop/HiveMain.java 
b/sharelib/hive/src/main/java/org/apache/oozie/action/hadoop/HiveMain.java
index 23231e3a3..f6e3e6482 100644
--- a/sharelib/hive/src/main/java/org/apache/oozie/action/hadoop/HiveMain.java
+++ b/sharelib/hive/src/main/java/org/apache/oozie/action/hadoop/HiveMain.java
@@ -38,6 +38,7 @@ import org.apache.hadoop.hive.cli.CliDriver;
 import org.apache.hadoop.hive.conf.HiveConf;
 
 import com.google.common.annotations.VisibleForTesting;
+import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
 
 public class HiveMain extends LauncherMain {
     @VisibleForTesting
@@ -182,6 +183,7 @@ public class HiveMain extends LauncherMain {
         return hiveConf;
     }
 
+    @SuppressFBWarnings(value = "PATH_TRAVERSAL_IN", justification = "Hive 
script file created without user input")
     @Override
     protected void run(String[] args) throws Exception {
         System.out.println();
@@ -310,6 +312,7 @@ public class HiveMain extends LauncherMain {
         CliDriver.main(args);
     }
 
+    @SuppressFBWarnings(value = "PATH_TRAVERSAL_IN", justification = "Hive 
script file created without user input")
     private static String readStringFromFile(String filePath) throws 
IOException {
         String line;
         BufferedReader br = null;

Reply via email to