Sqoop
  1. Sqoop
  2. SQOOP-875

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

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Trivial 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
          Cheolsoo Park added a comment -

          +1. Committed to sqoop2.

          Show
          Cheolsoo Park added a comment - +1. Committed to sqoop2.
          Hide
          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 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
          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 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

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Development