svn commit: r1152007 - /hive/trunk/metastore/src/test/org/apache/hadoop/hive/metastore/TestMetaStoreEventListener.java

[cw...@apache.org]

Author: cws
Date: Thu Jul 28 21:30:45 2011
New Revision: 1152007

URL: http://svn.apache.org/viewvc?rev=1152007&view=rev
Log:
HIVE-2276. Fix Inconsistency between RB and JIRA patches for HIVE-2194 (Sohan Jain via cws)

Modified:
    hive/trunk/metastore/src/test/org/apache/hadoop/hive/metastore/TestMetaStoreEventListener.java

Modified: hive/trunk/metastore/src/test/org/apache/hadoop/hive/metastore/TestMetaStoreEventListener.java
URL: http://svn.apache.org/viewvc/hive/trunk/metastore/src/test/org/apache/hadoop/hive/metastore/TestMetaStoreEventListener.java?rev=1152007&r1=1152006&r2=1152007&view=diff
==============================================================================
--- hive/trunk/metastore/src/test/org/apache/hadoop/hive/metastore/TestMetaStoreEventListener.java (original)
+++ hive/trunk/metastore/src/test/org/apache/hadoop/hive/metastore/TestMetaStoreEventListener.java Thu Jul 28 21:30:45 2011
@@ -19,7 +19,6 @@
 package org.apache.hadoop.hive.metastore;
 
 import java.util.HashMap;
-import java.util.Iterator;
 import java.util.List;
 import java.util.Map;
 
@@ -179,18 +178,8 @@ public class TestMetaStoreEventListener 
     List<FieldSchema> altCols = altTable.getSd().getCols();
     List<FieldSchema> newCols = altTable.getSd().getCols();
 
-    Iterator<FieldSchema> i = origCols.iterator();
-    Iterator<FieldSchema> j = oldCols.iterator();
-    while (i.hasNext()) {
-      assertEquals(i.next().getName(), j.next().getName());
-    }
-
-    i = altCols.iterator();
-    j = newCols.iterator();
-    while (i.hasNext()) {
-      String name = i.next().getName();
-      assertEquals(name, j.next().getName());
-    }
+    assertEquals(origCols, oldCols);
+    assertEquals(altCols, newCols);
 
     newTable = alterTableE.getNewTable();
     assertEquals(tbl.getDbName(), oldTable.getDbName());