Update comment.

nemccarthy · May 28, 2015 · 005649b · 005649b
1 parent 35d86f3
commit 005649b
Showing 1 changed file with 4 additions and 0 deletions.
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
@@ -721,13 +721,17 @@ private[hive] case class MetastoreRelation
   // to get Partition keys at here. We do this to make sure Hive will not try to use
   // any metastore utility functions. All of interactions between metastore and related
   // parts should be done through our ClientWrapper.
+  // Without the guard of ClientWrapper, we cannot touch metastore client (e.g. Hive class) and
+  // the conf associated with the metastore client (Hive.get().getConf()).
   /** PartitionKey attributes */
   val partitionKeys = hiveQlTable.getTTable.getPartitionKeys.map(_.toAttribute)
 
   // Since we create the TTable inside hiveQlTable manually, we can use TTbale's interface
   // to get non-partition columns at here. We do this to make sure Hive will not try to use
   // any metastore utility functions. All of interactions between metastore and related
   // parts should be done through our ClientWrapper.
+  // Without the guard of ClientWrapper, we cannot touch metastore client (e.g. Hive class) and
+  // the conf associated with the metastore client (Hive.get().getConf()).
   /** Non-partitionKey attributes */
   val attributes = hiveQlTable.getTTable.getSd.getCols.map(_.toAttribute)