You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by zs...@apache.org on 2008/12/08 21:11:59 UTC

svn commit: r724465 - in /hadoop/hive/trunk: CHANGES.txt metastore/src/test/org/apache/hadoop/hive/metastore/TestGetDBs.java metastore/src/test/org/apache/hadoop/hive/metastore/TestGetTables.java

Author: zshao
Date: Mon Dec  8 12:11:59 2008
New Revision: 724465

URL: http://svn.apache.org/viewvc?rev=724465&view=rev
Log:
HIVE-90. Fixed TestGetDBs for File.list() order. (Johan Oskarsson through zshao)

Modified:
    hadoop/hive/trunk/CHANGES.txt
    hadoop/hive/trunk/metastore/src/test/org/apache/hadoop/hive/metastore/TestGetDBs.java
    hadoop/hive/trunk/metastore/src/test/org/apache/hadoop/hive/metastore/TestGetTables.java

Modified: hadoop/hive/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/hive/trunk/CHANGES.txt?rev=724465&r1=724464&r2=724465&view=diff
==============================================================================
--- hadoop/hive/trunk/CHANGES.txt (original)
+++ hadoop/hive/trunk/CHANGES.txt Mon Dec  8 12:11:59 2008
@@ -27,6 +27,9 @@
 
   BUG FIXES
 
+    HIVE-90. Fixed TestGetDBs for File.list() order.
+    (Johan Oskarsson through zshao)
+
     HIVE-111. Support join without a ON clause. (Namit through zshao)
 
     HIVE-77. Thread safe query execution. (Joydeep through zshao)

Modified: hadoop/hive/trunk/metastore/src/test/org/apache/hadoop/hive/metastore/TestGetDBs.java
URL: http://svn.apache.org/viewvc/hadoop/hive/trunk/metastore/src/test/org/apache/hadoop/hive/metastore/TestGetDBs.java?rev=724465&r1=724464&r2=724465&view=diff
==============================================================================
--- hadoop/hive/trunk/metastore/src/test/org/apache/hadoop/hive/metastore/TestGetDBs.java (original)
+++ hadoop/hive/trunk/metastore/src/test/org/apache/hadoop/hive/metastore/TestGetDBs.java Mon Dec  8 12:11:59 2008
@@ -55,9 +55,9 @@
       DB.createDB("foo2", conf_);
       List<String> dbs = MetaStore.getDbs(conf_);
       assertTrue(dbs.size() == 3);
-      assertTrue(dbs.get(0).equals("foo1"));
-      assertTrue(dbs.get(1).equals("foo2"));
-      assertTrue(dbs.get(2).equals("default"));
+      assertTrue(dbs.contains("foo1"));
+      assertTrue(dbs.contains("foo2"));
+      assertTrue(dbs.contains("default"));
       cleanup();
     } catch(MetaException e) {
       e.printStackTrace();

Modified: hadoop/hive/trunk/metastore/src/test/org/apache/hadoop/hive/metastore/TestGetTables.java
URL: http://svn.apache.org/viewvc/hadoop/hive/trunk/metastore/src/test/org/apache/hadoop/hive/metastore/TestGetTables.java?rev=724465&r1=724464&r2=724465&view=diff
==============================================================================
--- hadoop/hive/trunk/metastore/src/test/org/apache/hadoop/hive/metastore/TestGetTables.java (original)
+++ hadoop/hive/trunk/metastore/src/test/org/apache/hadoop/hive/metastore/TestGetTables.java Mon Dec  8 12:11:59 2008
@@ -48,12 +48,12 @@
       List<String> tables = db.getTables(".+");
       assertTrue(tables.size() == 0);
     }
-    Table bar1 = Table.create(db, "bar1", createSchema("foo1","bar1"), conf_);
-    Table bar2 = Table.create(db, "bar2", createSchema("foo1","bar2"), conf_);
+    Table.create(db, "bar1", createSchema("foo1","bar1"), conf_);
+    Table.create(db, "bar2", createSchema("foo1","bar2"), conf_);
     List<String> tables = db.getTables(".+");
     assertTrue(tables.size() == 2);
-    assertTrue(tables.get(0).equals("bar1"));
-    assertTrue(tables.get(1).equals("bar2"));
+    assertTrue(tables.contains("bar1"));
+    assertTrue(tables.contains("bar2"));
     cleanup();
   }
 }