You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sqoop.apache.org by ch...@apache.org on 2013/02/06 23:14:24 UTC

git commit: SQOOP-860: NullPointerException when running procedure export against old database

Updated Branches:
  refs/heads/trunk bae5604a3 -> 666d5499a


SQOOP-860: NullPointerException when running procedure export against old database

(Jarek Jarcec Cecho via Cheolsoo Park)


Project: http://git-wip-us.apache.org/repos/asf/sqoop/repo
Commit: http://git-wip-us.apache.org/repos/asf/sqoop/commit/666d5499
Tree: http://git-wip-us.apache.org/repos/asf/sqoop/tree/666d5499
Diff: http://git-wip-us.apache.org/repos/asf/sqoop/diff/666d5499

Branch: refs/heads/trunk
Commit: 666d5499a2343c3489590deed3ef839e5532281d
Parents: bae5604
Author: Cheolsoo Park <ch...@apache.org>
Authored: Wed Feb 6 14:13:31 2013 -0800
Committer: Cheolsoo Park <ch...@apache.org>
Committed: Wed Feb 6 14:13:31 2013 -0800

----------------------------------------------------------------------
 src/java/org/apache/sqoop/manager/SqlManager.java |    7 +++----
 1 files changed, 3 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/sqoop/blob/666d5499/src/java/org/apache/sqoop/manager/SqlManager.java
----------------------------------------------------------------------
diff --git a/src/java/org/apache/sqoop/manager/SqlManager.java b/src/java/org/apache/sqoop/manager/SqlManager.java
index 03c9e64..f0a920d 100644
--- a/src/java/org/apache/sqoop/manager/SqlManager.java
+++ b/src/java/org/apache/sqoop/manager/SqlManager.java
@@ -199,10 +199,9 @@ public abstract class SqlManager
         results.close();
         getConnection().commit();
       }
-    } catch (SQLException sqlException) {
-      LOG.error("Error reading procedure metadata: "
-          + sqlException.toString());
-      return null;
+    } catch (SQLException e) {
+      LOG.error("Error reading procedure metadata: ", e);
+      throw new RuntimeException("Can't fetch column names for procedure.", e);
     }
   }