Uploaded image for project: 'Oozie'
  1. Oozie
  2. OOZIE-1987 Fix existing findbugs warnings
  3. OOZIE-2956

Fix Findbugs warnings related to reliance on default encoding in oozie-core

    XMLWordPrintableJSON

    Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 5.1.0
    • Component/s: build, core
    • Labels:
      None

      Description

      Currently Findbugs complains about the following warnings related to the reliance on the default encoding in the oozie-core module:

      [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]
      

      They should be fixed to get the code more reliable.

        Attachments

        1. OOZIE-2956-2.patch
          77 kB
          Jan Hentschel
        2. OOZIE-2956-1.patch
          77 kB
          Jan Hentschel
        3. OOZIE-2956-004.patch
          76 kB
          Kinga Marton
        4. OOZIE-2956-003.patch
          74 kB
          Kinga Marton

          Issue Links

            Activity

              People

              • Assignee:
                kmarton Kinga Marton
                Reporter:
                janh Jan Hentschel
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: