Uploaded image for project: 'Sqoop'
  1. Sqoop
  2. SQOOP-875

Sqoop2: ClassUtils.instantiate class should not be logging on ERROR level

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Trivial
    • Resolution: Fixed
    • Affects Version/s: 1.99.1
    • Fix Version/s: 1.99.2
    • Component/s: None
    • Labels:
      None

      Description

      ClassUtils was designed to handle gracefully all error conditions and it's return null on every issue. Method loadClass already logs on debug level, but method instantiate logs on ERROR level. I believe that we should change that to debug level as well.

      1. bugSQOOP-875.patch
        1 kB
        Jarek Jarcec Cecho

        Issue Links

          Activity

          Hide
          hudson Hudson added a comment -

          Integrated in Sqoop2-hadoop200 #187 (See https://builds.apache.org/job/Sqoop2-hadoop200/187/)
          SQOOP-875: ClassUtils.instantiate class should not be logging on ERROR level (Revision e8ca6f6c93cf193bd6842746e4e9df5d9f85f8d5)

          Result = SUCCESS
          cheolsoo : https://git-wip-us.apache.org/repos/asf?p=sqoop.git&a=commit&h=e8ca6f6c93cf193bd6842746e4e9df5d9f85f8d5
          Files :

          • common/src/main/java/org/apache/sqoop/utils/ClassUtils.java
          Show
          hudson Hudson added a comment - Integrated in Sqoop2-hadoop200 #187 (See https://builds.apache.org/job/Sqoop2-hadoop200/187/ ) SQOOP-875 : ClassUtils.instantiate class should not be logging on ERROR level (Revision e8ca6f6c93cf193bd6842746e4e9df5d9f85f8d5) Result = SUCCESS cheolsoo : https://git-wip-us.apache.org/repos/asf?p=sqoop.git&a=commit&h=e8ca6f6c93cf193bd6842746e4e9df5d9f85f8d5 Files : common/src/main/java/org/apache/sqoop/utils/ClassUtils.java
          Hide
          hudson Hudson added a comment -

          Integrated in Sqoop2-hadoop100 #174 (See https://builds.apache.org/job/Sqoop2-hadoop100/174/)
          SQOOP-875: ClassUtils.instantiate class should not be logging on ERROR level (Revision e8ca6f6c93cf193bd6842746e4e9df5d9f85f8d5)

          Result = SUCCESS
          cheolsoo : https://git-wip-us.apache.org/repos/asf?p=sqoop.git&a=commit&h=e8ca6f6c93cf193bd6842746e4e9df5d9f85f8d5
          Files :

          • common/src/main/java/org/apache/sqoop/utils/ClassUtils.java
          Show
          hudson Hudson added a comment - Integrated in Sqoop2-hadoop100 #174 (See https://builds.apache.org/job/Sqoop2-hadoop100/174/ ) SQOOP-875 : ClassUtils.instantiate class should not be logging on ERROR level (Revision e8ca6f6c93cf193bd6842746e4e9df5d9f85f8d5) Result = SUCCESS cheolsoo : https://git-wip-us.apache.org/repos/asf?p=sqoop.git&a=commit&h=e8ca6f6c93cf193bd6842746e4e9df5d9f85f8d5 Files : common/src/main/java/org/apache/sqoop/utils/ClassUtils.java
          Hide
          cheolsoo Cheolsoo Park added a comment -

          +1. Committed to sqoop2.

          Show
          cheolsoo Cheolsoo Park added a comment - +1. Committed to sqoop2.

            People

            • Assignee:
              jarcec Jarek Jarcec Cecho
              Reporter:
              jarcec Jarek Jarcec Cecho
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development