You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by nz...@apache.org on 2011/03/23 18:56:38 UTC

svn commit: r1084653 - in /hive/trunk/jdbc/src: java/org/apache/hadoop/hive/jdbc/HiveMetaDataResultSet.java test/org/apache/hadoop/hive/jdbc/TestJdbcDriver.java

Author: nzhang
Date: Wed Mar 23 17:56:38 2011
New Revision: 1084653

URL: http://svn.apache.org/viewvc?rev=1084653&view=rev
Log:
HIVE-2069. NullPointerException on getSchemas (Bennie Schut via Ning Zhang)

Modified:
    hive/trunk/jdbc/src/java/org/apache/hadoop/hive/jdbc/HiveMetaDataResultSet.java
    hive/trunk/jdbc/src/test/org/apache/hadoop/hive/jdbc/TestJdbcDriver.java

Modified: hive/trunk/jdbc/src/java/org/apache/hadoop/hive/jdbc/HiveMetaDataResultSet.java
URL: http://svn.apache.org/viewvc/hive/trunk/jdbc/src/java/org/apache/hadoop/hive/jdbc/HiveMetaDataResultSet.java?rev=1084653&r1=1084652&r2=1084653&view=diff
==============================================================================
--- hive/trunk/jdbc/src/java/org/apache/hadoop/hive/jdbc/HiveMetaDataResultSet.java (original)
+++ hive/trunk/jdbc/src/java/org/apache/hadoop/hive/jdbc/HiveMetaDataResultSet.java Wed Mar 23 17:56:38 2011
@@ -29,9 +29,21 @@ public abstract class HiveMetaDataResult
   public HiveMetaDataResultSet(final List<String> columnNames
           , final List<String> columnTypes
           , final List<M> data) throws SQLException {
-    this.data = new ArrayList<M>(data);
-    this.columnNames = new ArrayList<String>(columnNames);
-    this.columnTypes = new ArrayList<String>(columnTypes);
+    if (data!=null) {
+      this.data = new ArrayList<M>(data);
+    } else {
+      this.data =  new ArrayList<M>();
+    }
+    if (columnNames!=null) {
+      this.columnNames = new ArrayList<String>(columnNames);
+    } else {
+      this.columnNames =  new ArrayList<String>();
+    }
+    if (columnTypes!=null) {
+      this.columnTypes = new ArrayList<String>(columnTypes);
+    } else {
+      this.columnTypes =  new ArrayList<String>();
+    }
   }
 
   @Override

Modified: hive/trunk/jdbc/src/test/org/apache/hadoop/hive/jdbc/TestJdbcDriver.java
URL: http://svn.apache.org/viewvc/hive/trunk/jdbc/src/test/org/apache/hadoop/hive/jdbc/TestJdbcDriver.java?rev=1084653&r1=1084652&r2=1084653&view=diff
==============================================================================
--- hive/trunk/jdbc/src/test/org/apache/hadoop/hive/jdbc/TestJdbcDriver.java (original)
+++ hive/trunk/jdbc/src/test/org/apache/hadoop/hive/jdbc/TestJdbcDriver.java Wed Mar 23 17:56:38 2011
@@ -481,7 +481,17 @@ public class TestJdbcDriver extends Test
       cnt++;
     }
     rs.close();
-    assertEquals("Incorrect schema count", 1, cnt);
+    assertEquals("Incorrect catalog count", 1, cnt);
+  }
+
+  public void testMetaDataGetSchemas() throws SQLException {
+    ResultSet rs = (ResultSet)con.getMetaData().getSchemas();
+    int cnt = 0;
+    while (rs.next()) {
+      cnt++;
+    }
+    rs.close();
+    assertEquals("Incorrect schema count", 0, cnt);
   }
 
   public void testMetaDataGetTableTypes() throws SQLException {