[ 
https://issues.apache.org/jira/browse/OOZIE-3121?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16350551#comment-16350551
 ] 

Hadoop QA commented on OOZIE-3121:
----------------------------------


Testing JIRA OOZIE-3121

Cleaning local git workspace

----------------------------

{color:green}+1 PATCH_APPLIES{color}
{color:green}+1 CLEAN{color}
{color:red}-1 RAW_PATCH_ANALYSIS{color}
.    {color:green}+1{color} the patch does not introduce any @author tags
.    {color:green}+1{color} the patch does not introduce any tabs
.    {color:green}+1{color} the patch does not introduce any trailing spaces
.    {color:green}+1{color} the patch does not introduce any line longer than 
132
.    {color:red}-1{color} the patch does not add/modify any testcase
{color:green}+1 RAT{color}
.    {color:green}+1{color} the patch does not seem to introduce new RAT 
warnings
{color:green}+1 JAVADOC{color}
.    {color:green}+1{color} the patch does not seem to introduce new Javadoc 
warnings
.    {color:red}WARNING{color}: the current HEAD has 100 Javadoc warning(s)
{color:green}+1 COMPILE{color}
.    {color:green}+1{color} HEAD compiles
.    {color:green}+1{color} patch compiles
.    {color:green}+1{color} the patch does not seem to introduce new javac 
warnings
{color:red}-1{color} There are [126] new bugs found below threshold in total 
that must be fixed.
. {color:green}+1{color} There are no new bugs found in [docs].
. {color:green}+1{color} There are no new bugs found in [sharelib/distcp].
. {color:red}-1{color} There are [2] new bugs found below threshold in 
[sharelib/hive] that must be fixed.
. You can find the FindBugs diff here (look for the red and orange ones): 
sharelib/hive/findbugs-new.html
. The most important FindBugs errors are:
. At HiveMain.java:[line 317]: 
java/io/FileInputStream.<init>(Ljava/lang/String;)V reads a file whose 
location might be specified by user input
. At HiveMain.java:[line 226]: At HiveMain.java:[line 207]
. At HiveMain.java:[line 210]: java/io/File.<init>(Ljava/lang/String;)V 
reads a file whose location might be specified by user input
. At HiveMain.java:[line 207]
. {color:red}-1{color} There are [2] new bugs found below threshold in 
[sharelib/spark] that must be fixed.
. You can find the FindBugs diff here (look for the red and orange ones): 
sharelib/spark/findbugs-new.html
. The most important FindBugs errors are:
. At SparkMain.java:[line 128]: 
java/io/File.<init>(Ljava/io/File;Ljava/lang/String;)V reads a file whose 
location might be specified by user input
. java/io/File.<init>(Ljava/lang/String;)V reads a file whose location 
might be specified by user input: At SparkMain.java:[line 128]
. At SparkMain.java:[line 161]: At SparkMain.java:[line 168]
. At SparkMain.java:[line 167]
. {color:red}-1{color} There are [3] new bugs found below threshold in 
[sharelib/hive2] that must be fixed.
. You can find the FindBugs diff here (look for the red and orange ones): 
sharelib/hive2/findbugs-new.html
. The most important FindBugs errors are:
. At Hive2Main.java:[line 279]: 
java/io/FileInputStream.<init>(Ljava/lang/String;)V reads a file whose 
location might be specified by user input
. At Hive2Main.java:[line 165]: At Hive2Main.java:[line 146]
. At Hive2Main.java:[line 149]: java/io/File.<init>(Ljava/lang/String;)V 
reads a file whose location might be specified by user input
. java/io/FileOutputStream.<init>(Ljava/lang/String;)V writes to a file 
whose location might be specified by user input: At Hive2Main.java:[line 146]
. At Hive2Main.java:[line 118]: At Hive2Main.java:[line 267]
. {color:green}+1{color} There are no new bugs found in [sharelib/hcatalog].
. {color:green}+1{color} There are no new bugs found in [sharelib/streaming].
. {color:red}-1{color} There are [3] new bugs found below threshold in 
[sharelib/pig] that must be fixed.
. You can find the FindBugs diff here (look for the red and orange ones): 
sharelib/pig/findbugs-new.html
. The most important FindBugs errors are:
. At PigMain.java:[line 258]: java/io/File.<init>(Ljava/lang/String;)V 
reads a file whose location might be specified by user input
. At PigMain.java:[line 141]: At PigMain.java:[line 131]
. At PigMain.java:[line 245]: At PigMain.java:[line 199]
. At PigMain.java:[line 137]: java/io/File.<init>(Ljava/lang/String;)V 
reads a file whose location might be specified by user input
. java/io/File.<init>(Ljava/lang/String;)V reads a file whose location 
might be specified by user input: At PigMain.java:[line 131]
. {color:green}+1{color} There are no new bugs found in [sharelib/sqoop].
. {color:red}-1{color} There are [9] new bugs found below threshold in 
[sharelib/oozie] that must be fixed, listing only the first [5] ones.
. You can find the FindBugs diff here (look for the red and orange ones): 
sharelib/oozie/findbugs-new.html
. The top [5] most important FindBugs errors are:
. At LauncherMain.java:[line 473]: 
java/io/File.<init>(Ljava/lang/String;)V reads a file whose location 
might be specified by user input
. At LauncherMain.java:[line 463]: 
java/io/File.<init>(Ljava/lang/String;)V reads a file whose location 
might be specified by user input
. At LauncherMain.java:[line 140]: 
java/io/File.<init>(Ljava/lang/String;)V reads a file whose location 
might be specified by user input
. java/io/FileReader.<init>(Ljava/lang/String;)V reads a file whose 
location might be specified by user input: At LauncherMain.java:[line 176]
. At LauncherMain.java:[line 176]: At LauncherMain.java:[line 144]
. {color:red}-1{color} There are [2] new bugs found below threshold in 
[examples] that must be fixed.
. You can find the FindBugs diff here (look for the red and orange ones): 
examples/findbugs-new.html
. The most important FindBugs errors are:
. At LocalOozieExample.java:[line 47]: 
java/io/File.<init>(Ljava/lang/String;)V reads a file whose location 
might be specified by user input
. java/io/FileInputStream.<init>(Ljava/lang/String;)V reads a file whose 
location might be specified by user input: At LocalOozieExample.java:[line 35]
. At LocalOozieExample.java:[line 35]: At LocalOozieExample.java:[line 72]
. {color:red}-1{color} There are [8] new bugs found below threshold in [client] 
that must be fixed, listing only the first [5] ones.
. You can find the FindBugs diff here (look for the red and orange ones): 
client/findbugs-new.html
. The top [5] most important FindBugs errors are:
. At OozieCLI.java:[line 841]: java/io/File.<init>(Ljava/lang/String;)V 
reads a file whose location might be specified by user input
. java/io/FileInputStream.<init>(Ljava/lang/String;)V reads a file whose 
location might be specified by user input: At OozieCLI.java:[line 839]
. At OozieCLI.java:[line 839]: At OozieCLI.java:[line 849]
. At OozieCLI.java:[line 876]: java/io/File.<init>(Ljava/lang/String;)V 
reads a file whose location might be specified by user input
. java/io/File.<init>(Ljava/lang/String;)V reads a file whose location 
might be specified by user input: At OozieCLI.java:[line 871]
. {color:red}-1{color} There are [67] new bugs found below threshold in [core] 
that must be fixed, listing only the first [5] ones.
. You can find the FindBugs diff here (look for the red and orange ones): 
core/findbugs-new.html
. The top [5] most important FindBugs errors are:
. At EmailActionExecutor.java:[line 268]: 
java/io/File.<init>(Ljava/lang/String;)V reads a file whose location 
might be specified by user input
. At EmailActionExecutor.java:[line 163]: At EmailActionExecutor.java:[line 161]
. At EmailActionExecutor.java:[line 177]: At EmailActionExecutor.java:[line 172]
. At SshActionExecutor.java:[line 135]: This usage of 
java/lang/Runtime.exec([Ljava/lang/String;)Ljava/lang/Process; can be 
vulnerable to Command Injection
. At SshActionExecutor.java:[line 132]: At SshActionExecutor.java:[line 131]
. {color:red}-1{color} There are [29] new bugs found below threshold in [tools] 
that must be fixed, listing only the first [5] ones.
. You can find the FindBugs diff here (look for the red and orange ones): 
tools/findbugs-new.html
. The top [5] most important FindBugs errors are:
. At OozieDBCLI.java:[line 548]: 
java/io/FileWriter.<init>(Ljava/lang/String;Z)V writes to a file whose 
location might be specified by user input
. At OozieDBCLI.java:[line 126]: At OozieDBCLI.java:[line 122]
. At OozieDBCLI.java:[line 229]: At OozieDBCLI.java:[line 133]
. At OozieDBCLI.java:[line 703]: At OozieDBCLI.java:[line 249]
. At OozieDBCLI.java:[line 577]: 
java/io/FileWriter.<init>(Ljava/lang/String;Z)V writes to a file whose 
location might be specified by user input
. {color:red}-1{color} There are [1] new bugs found below threshold in [server] 
that must be fixed.
. You can find the FindBugs diff here (look for the red and orange ones): 
server/findbugs-new.html
. The most important FindBugs errors are:
. At JspHandlerProvider.java:[line 43]: 
java/io/File.<init>(Ljava/lang/String;Ljava/lang/String;)V reads a file 
whose location might be specified by user input
. At JspHandlerProvider.java:[line 43]
{color:green}+1 BACKWARDS_COMPATIBILITY{color}
.    {color:green}+1{color} the patch does not change any JPA 
Entity/Colum/Basic/Lob/Transient annotations
.    {color:green}+1{color} the patch does not modify JPA files
{color:red}-1 TESTS{color}
.    Tests run: 2088
.    Tests failed: 4
.    Tests errors: 1

.    The patch failed the following testcases:

testPartitionDependency(org.apache.oozie.service.TestPartitionDependencyManagerService)
testCoordActionRecoveryServiceForWaitingRegisterPartition(org.apache.oozie.service.TestRecoveryService)
testConnectionDrop(org.apache.oozie.jms.TestJMSJobEventListener)
testPartitionDependency(org.apache.oozie.service.TestPartitionDependencyManagerEhcache)

.    Tests failing with errors:
testConnectionRetry(org.apache.oozie.service.TestJMSAccessorService)

.    {color:orange}Tests failed at first run:{color}
TestJavaActionExecutor#testCredentialsSkip
.    For the complete list of flaky tests, see TEST-SUMMARY-FULL files.
{color:green}+1 DISTRO{color}
.    {color:green}+1{color} distro tarball builds with the patch 

----------------------------
{color:red}*-1 Overall result, please check the reported -1(s)*{color}

{color:red}. There is at least one warning, please check{color}

The full output of the test-patch run is available at

. https://builds.apache.org/job/PreCommit-OOZIE-Build/368/



> bump all maven plugins to latest versions
> -----------------------------------------
>
>                 Key: OOZIE-3121
>                 URL: https://issues.apache.org/jira/browse/OOZIE-3121
>             Project: Oozie
>          Issue Type: Improvement
>    Affects Versions: 4.3.0
>            Reporter: Artem Ervits
>            Assignee: Artem Ervits
>            Priority: Major
>             Fix For: 5.0.0
>
>         Attachments: OOZIE-3121-02.patch, OOZIE-3121.patch
>
>
> bump all maven plugins to latest



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to