Repository: spark
Updated Branches:
  refs/heads/master 10f1f1965 -> 34ebcc6b5


[MINOR] Improve documentation for HiveStringType's

The diff should be self-explanatory.

Author: Reynold Xin <r...@databricks.com>

Closes #21897 from rxin/hivestringtypedoc.


Project: http://git-wip-us.apache.org/repos/asf/spark/repo
Commit: http://git-wip-us.apache.org/repos/asf/spark/commit/34ebcc6b
Tree: http://git-wip-us.apache.org/repos/asf/spark/tree/34ebcc6b
Diff: http://git-wip-us.apache.org/repos/asf/spark/diff/34ebcc6b

Branch: refs/heads/master
Commit: 34ebcc6b5246c1a47e6d3b2dbb23e368de25219e
Parents: 10f1f19
Author: Reynold Xin <r...@databricks.com>
Authored: Fri Jul 27 15:34:06 2018 -0700
Committer: Xiao Li <gatorsm...@gmail.com>
Committed: Fri Jul 27 15:34:06 2018 -0700

----------------------------------------------------------------------
 .../scala/org/apache/spark/sql/types/HiveStringType.scala    | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/spark/blob/34ebcc6b/sql/catalyst/src/main/scala/org/apache/spark/sql/types/HiveStringType.scala
----------------------------------------------------------------------
diff --git 
a/sql/catalyst/src/main/scala/org/apache/spark/sql/types/HiveStringType.scala 
b/sql/catalyst/src/main/scala/org/apache/spark/sql/types/HiveStringType.scala
index e0bca93..4eb3226 100644
--- 
a/sql/catalyst/src/main/scala/org/apache/spark/sql/types/HiveStringType.scala
+++ 
b/sql/catalyst/src/main/scala/org/apache/spark/sql/types/HiveStringType.scala
@@ -56,14 +56,18 @@ object HiveStringType {
 }
 
 /**
- * Hive char type.
+ * Hive char type. Similar to other HiveStringType's, these datatypes should 
only used for
+ * parsing, and should NOT be used anywhere else. Any instance of these data 
types should be
+ * replaced by a [[StringType]] before analysis.
  */
 case class CharType(length: Int) extends HiveStringType {
   override def simpleString: String = s"char($length)"
 }
 
 /**
- * Hive varchar type.
+ * Hive varchar type. Similar to other HiveStringType's, these datatypes 
should only used for
+ * parsing, and should NOT be used anywhere else. Any instance of these data 
types should be
+ * replaced by a [[StringType]] before analysis.
  */
 case class VarcharType(length: Int) extends HiveStringType {
   override def simpleString: String = s"varchar($length)"


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org
For additional commands, e-mail: commits-h...@spark.apache.org

Reply via email to