Don't use reflection to read options

marmbrus · yhuai · commit c00bb1bf25b8 · 2015-01-08T13:38:32.000-08:00
diff --git a/sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveMetastoreCatalog.scala b/sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveMetastoreCatalog.scala
@@ -73,15 +73,13 @@ private[hive] class HiveMetastoreCatalog(hive: HiveContext) extends Catalog with
 
           // It does not appear that the ql client for the metastore has a way to enumerate all the
           // SerDe properties directly...
-          val method = classOf[Table].getDeclaredMethod("getSerdeInfo")
-          method.setAccessible(true)
-          val serdeInfo = method.invoke(table).asInstanceOf[SerDeInfo]
+          val options = table.getTTable.getSd.getSerdeInfo.getParameters.toMap
 
           val resolvedRelation =
             ResolvedDataSource(
               hive,
               table.getProperty("spark.sql.sources.provider"),
-              serdeInfo.getParameters.toMap)
+              options)
 
           LogicalRelation(resolvedRelation.relation)
         }
