[ https://issues.apache.org/jira/browse/OOZIE-2956?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16064653#comment-16064653 ]
Hadoop QA commented on OOZIE-2956: ---------------------------------- Testing JIRA OOZIE-2956 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 1 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:green}+1{color} There are no new bugs found in total. {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: 1968 . Tests failed: 0 . Tests errors: 63 . The patch failed the following testcases: . . Tests failing with errors: . testAuthorizationServiceForBundle(org.apache.oozie.service.TestAuthorizationService) . testAdminUsersWithAdminFile(org.apache.oozie.service.TestAuthorizationService) . testAdminUsersWithAdminGroup(org.apache.oozie.service.TestAuthorizationService) . testAuthorizationServiceForCoord(org.apache.oozie.service.TestAuthorizationService) . testAuthorizationServiceUseDefaultGroup(org.apache.oozie.service.TestAuthorizationService) . testDefaultGroup(org.apache.oozie.service.TestAuthorizationService) . testErrors(org.apache.oozie.service.TestAuthorizationService) . testAuthorizationServiceUseACLs(org.apache.oozie.service.TestAuthorizationService) . testRerunFork(org.apache.oozie.command.wf.TestReRunXCommand) . testPurgeMissingDependencies(org.apache.oozie.service.TestHAPartitionDependencyManagerEhCache) . testDependencyCacheWithHA(org.apache.oozie.service.TestHAPartitionDependencyManagerEhCache) . testCheckAfterActionDelete(org.apache.oozie.service.TestHAPartitionDependencyManagerEhCache) . testActionInputCheckLatestActionCreationTimeWithPushDependency(org.apache.oozie.command.coord.TestCoordActionInputCheckXCommandNonUTC) . testActionInputCheckLatestCurrentTimeWithPushDependency(org.apache.oozie.command.coord.TestCoordActionInputCheckXCommandNonUTC) . testHCatCanonicalHostName(org.apache.oozie.service.TestPartitionDependencyManagerService) . testActionCheckerService(org.apache.oozie.service.TestActionCheckerService) . testActionCheckerServiceCoord(org.apache.oozie.service.TestActionCheckerService) . testActionCheckerServiceDelay(org.apache.oozie.service.TestActionCheckerService) . testEndError(org.apache.oozie.command.wf.TestActionErrors) . testStartNonTransientWithCoordActionUpdate(org.apache.oozie.command.wf.TestActionErrors) . testKillNodeErrorMessageError(org.apache.oozie.command.wf.TestActionErrors) . testEndDataNotSet(org.apache.oozie.command.wf.TestActionErrors) . testStartErrorWithUserRetry(org.apache.oozie.command.wf.TestActionErrors) . testKillNodeErrorMessageError2(org.apache.oozie.command.wf.TestActionErrors) . testStartTransient(org.apache.oozie.command.wf.TestActionErrors) . testKillNodeErrorMessage(org.apache.oozie.command.wf.TestActionErrors) . testStartNonTransient(org.apache.oozie.command.wf.TestActionErrors) . testEndNonTransient(org.apache.oozie.command.wf.TestActionErrors) . testExecutionDataNotSet(org.apache.oozie.command.wf.TestActionErrors) . testEndErrorWithUserRetry(org.apache.oozie.command.wf.TestActionErrors) . testEndNonTransientWithCoordActionUpdate(org.apache.oozie.command.wf.TestActionErrors) . testEndTransient(org.apache.oozie.command.wf.TestActionErrors) . testStartError(org.apache.oozie.command.wf.TestActionErrors) . testActionInputCheckLatestActionCreationTimeWithPushDependency(org.apache.oozie.command.coord.TestCoordActionInputCheckXCommand) . testActionInputCheckLatestCurrentTimeWithPushDependency(org.apache.oozie.command.coord.TestCoordActionInputCheckXCommand) . testPurgeMissingDependencies(org.apache.oozie.service.TestHAPartitionDependencyManagerService) . testDependencyCacheWithHA(org.apache.oozie.service.TestHAPartitionDependencyManagerService) . testCheckAfterActionDelete(org.apache.oozie.service.TestHAPartitionDependencyManagerService) . testRetryConsoleUrl(org.apache.oozie.command.wf.TestWorkflowActionRetryInfoXCommand) . testActionCheckErrorUserRetry(org.apache.oozie.command.wf.TestActionCheckXCommand) . testActionCheckErrorNoUserRetry(org.apache.oozie.command.wf.TestActionCheckXCommand) . testWfFailure(org.apache.oozie.command.wf.TestForkedActionStartXCommand) . testWfSuccess(org.apache.oozie.command.wf.TestForkedActionStartXCommand) . testCoordActionRecoveryServiceForSuspended(org.apache.oozie.service.TestRecoveryService) . testCoordCreateNotifyParentFailed(org.apache.oozie.service.TestRecoveryService) . testWorkflowActionRecoveryService(org.apache.oozie.service.TestRecoveryService) . testCoordActionRecoveryServiceForWaitingRegisterPartition(org.apache.oozie.service.TestRecoveryService) . testBundleRecoveryCoordCreate(org.apache.oozie.service.TestRecoveryService) . testCoordActionRecoveryServiceForKilled(org.apache.oozie.service.TestRecoveryService) . testBundleRecoveryCoordExists(org.apache.oozie.service.TestRecoveryService) . testCoordActionRecoveryServiceForResume(org.apache.oozie.service.TestRecoveryService) . testCoordActionRecoveryServiceForSubmitted(org.apache.oozie.service.TestRecoveryService) . testCoordActionRecoveryServiceForWaiting(org.apache.oozie.service.TestRecoveryService) . testWorkflowActionRecoveryUserRetry(org.apache.oozie.service.TestRecoveryService) . testPurgeServiceForBundle(org.apache.oozie.service.TestPurgeService) . testPurgeServiceForCoordinator(org.apache.oozie.service.TestPurgeService) . testPurgeServiceForWorkflow(org.apache.oozie.service.TestPurgeService) . testJobDefinition(org.apache.oozie.TestDagEngine) . testGetJobs(org.apache.oozie.TestDagEngine) . testSubmit(org.apache.oozie.TestDagEngine) . testHCatCanonicalHostName(org.apache.oozie.service.TestPartitionDependencyManagerEhcache) . testUserRetryPolicy(org.apache.oozie.command.wf.TestActionUserRetry) . testUserRetry(org.apache.oozie.command.wf.TestActionUserRetry) {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/oozie-trunk-precommit-build/3926/ > Fix Findbugs warnings related to reliance on default encoding in oozie-core > --------------------------------------------------------------------------- > > Key: OOZIE-2956 > URL: https://issues.apache.org/jira/browse/OOZIE-2956 > Project: Oozie > Issue Type: Sub-task > Components: build, core > Reporter: Jan Hentschel > Assignee: Jan Hentschel > Attachments: OOZIE-2956-1.patch > > > Currently Findbugs complains about the following warnings related to the > reliance on the default encoding in the *oozie-core* module: > {code} > [INFO] Found reliance on default encoding in > org.apache.oozie.StringBlob.getRawBlob(): String.getBytes() > [org.apache.oozie.StringBlob] At StringBlob.java:[line 119] > [INFO] Found reliance on default encoding in > org.apache.oozie.action.ActionExecutor.registerError(String, > ActionExecutorException$ErrorType, String): > java.io.ByteArrayOutputStream.toString() > [org.apache.oozie.action.ActionExecutor] At ActionExecutor.java:[line 354] > [INFO] Found reliance on default encoding in > org.apache.oozie.action.ActionExecutor.registerError(String, > ActionExecutorException$ErrorType, String): new > java.io.PrintStream(OutputStream) [org.apache.oozie.action.ActionExecutor] At > ActionExecutor.java:[line 353] > [INFO] Found reliance on default encoding in > org.apache.oozie.action.hadoop.LauncherHelper.getRecoveryId(Configuration, > Path, String): new java.io.InputStreamReader(InputStream) > [org.apache.oozie.action.hadoop.LauncherHelper] At LauncherHelper.java:[line > 69] > [INFO] Found reliance on default encoding in > org.apache.oozie.action.hadoop.LauncherHelper.getTag(String): > String.getBytes() [org.apache.oozie.action.hadoop.LauncherHelper] At > LauncherHelper.java:[line 160] > [INFO] Found reliance on default encoding in > org.apache.oozie.action.hadoop.LauncherHelper$1.run(): new > java.io.InputStreamReader(InputStream) > [org.apache.oozie.action.hadoop.LauncherHelper$1, > org.apache.oozie.action.hadoop.LauncherHelper$1, > org.apache.oozie.action.hadoop.LauncherHelper$1, > org.apache.oozie.action.hadoop.LauncherHelper$1, > org.apache.oozie.action.hadoop.LauncherHelper$1] At LauncherHelper.java:[line > 271]Another occurrence at LauncherHelper.java:[line 277]Another occurrence at > LauncherHelper.java:[line 285]Another occurrence at LauncherHelper.java:[line > 293]Another occurrence at LauncherHelper.java:[line 299] > [INFO] Found reliance on default encoding in > org.apache.oozie.action.hadoop.MapReduceActionExecutor.end(ActionExecutor$Context, > WorkflowAction): String.getBytes() > [org.apache.oozie.action.hadoop.MapReduceActionExecutor] At > MapReduceActionExecutor.java:[line 250] > [INFO] Found reliance on default encoding in > org.apache.oozie.action.hadoop.SqoopActionExecutor.end(ActionExecutor$Context, > WorkflowAction): String.getBytes() > [org.apache.oozie.action.hadoop.SqoopActionExecutor] At > SqoopActionExecutor.java:[line 186] > [INFO] Found reliance on default encoding in > org.apache.oozie.action.ssh.SshActionExecutor.drainBuffers(Process, > StringBuffer, StringBuffer, int): new java.io.InputStreamReader(InputStream) > [org.apache.oozie.action.ssh.SshActionExecutor, > org.apache.oozie.action.ssh.SshActionExecutor] At > SshActionExecutor.java:[line 684]Another occurrence at > SshActionExecutor.java:[line 685] > [INFO] Found reliance on default encoding in > org.apache.oozie.action.ssh.SshActionExecutor.initSshScripts(): new > java.io.FileWriter(String) [org.apache.oozie.action.ssh.SshActionExecutor, > org.apache.oozie.action.ssh.SshActionExecutor] At > SshActionExecutor.java:[line 502]Another occurrence at > SshActionExecutor.java:[line 504] > [INFO] Found reliance on default encoding in > org.apache.oozie.command.bundle.BundleSubmitXCommand.readDefinition(String): > new java.io.InputStreamReader(InputStream) > [org.apache.oozie.command.bundle.BundleSubmitXCommand] At > BundleSubmitXCommand.java:[line 322] > [INFO] Found reliance on default encoding in > org.apache.oozie.command.coord.CoordSubmitXCommand.readDefinition(String): > new java.io.InputStreamReader(InputStream) > [org.apache.oozie.command.coord.CoordSubmitXCommand] At > CoordSubmitXCommand.java:[line 1193] > [INFO] Found reliance on default encoding in > org.apache.oozie.command.coord.CoordUpdateXCommand.getDiffinGitFormat(String, > String): java.io.ByteArrayOutputStream.toString() > [org.apache.oozie.command.coord.CoordUpdateXCommand] At > CoordUpdateXCommand.java:[line 203] > [INFO] Found reliance on default encoding in > org.apache.oozie.command.coord.CoordUpdateXCommand.getDiffinGitFormat(String, > String): String.getBytes() > [org.apache.oozie.command.coord.CoordUpdateXCommand, > org.apache.oozie.command.coord.CoordUpdateXCommand] At > CoordUpdateXCommand.java:[line 198]Another occurrence at > CoordUpdateXCommand.java:[line 199] > [INFO] Found reliance on default encoding in > org.apache.oozie.service.AuthorizationService.loadAdminUsers(): new > java.io.InputStreamReader(InputStream) > [org.apache.oozie.service.AuthorizationService] At > AuthorizationService.java:[line 172] > [INFO] Found reliance on default encoding in > org.apache.oozie.service.WorkflowAppService.readDefinition(String, String, > Configuration): new java.io.InputStreamReader(InputStream) > [org.apache.oozie.service.WorkflowAppService] At > WorkflowAppService.java:[line 132] > [INFO] Found reliance on default encoding in > org.apache.oozie.servlet.V1JobServlet.getJob(HttpServletRequest, > HttpServletResponse): new String(byte[]) > [org.apache.oozie.servlet.V1JobServlet] At V1JobServlet.java:[line 223] > [INFO] Found reliance on default encoding in > org.apache.oozie.servlet.V2ValidateServlet.doPost(HttpServletRequest, > HttpServletResponse): new java.io.InputStreamReader(InputStream) > [org.apache.oozie.servlet.V2ValidateServlet, > org.apache.oozie.servlet.V2ValidateServlet] At V2ValidateServlet.java:[line > 84]Another occurrence at V2ValidateServlet.java:[line 92] > [INFO] Found reliance on default encoding in > org.apache.oozie.util.AuthUrlClient$1.run(): new > java.io.InputStreamReader(InputStream) > [org.apache.oozie.util.AuthUrlClient$1] At AuthUrlClient.java:[line 136] > [INFO] Found reliance on default encoding in > org.apache.oozie.util.IOUtils.getResourceAsReader(String, int): new > java.io.InputStreamReader(InputStream) [org.apache.oozie.util.IOUtils] At > IOUtils.java:[line 119] > [INFO] Found reliance on default encoding in > org.apache.oozie.util.IOUtils.getResourceAsString(String, int): new > java.io.InputStreamReader(InputStream) [org.apache.oozie.util.IOUtils] At > IOUtils.java:[line 136] > [INFO] Found reliance on default encoding in > org.apache.oozie.util.MultiFileReader.openNextReader(): new > java.io.FileReader(File) [org.apache.oozie.util.MultiFileReader] At > MultiFileReader.java:[line 86] > [INFO] Found reliance on default encoding in > org.apache.oozie.util.MultiFileReader.openNextReader(): new > java.io.InputStreamReader(InputStream) > [org.apache.oozie.util.MultiFileReader] At MultiFileReader.java:[line 82] > [INFO] Found reliance on default encoding in > org.apache.oozie.util.XConfiguration.toXmlString(boolean): new String(byte[]) > [org.apache.oozie.util.XConfiguration] At XConfiguration.java:[line 368] > [INFO] Found reliance on default encoding in > org.apache.oozie.util.XmlUtils.validateXml(Schema, String): String.getBytes() > [org.apache.oozie.util.XmlUtils] At XmlUtils.java:[line 284] > {code} > They should be fixed to get the code more reliable. -- This message was sent by Atlassian JIRA (v6.4.14#64029)