Make methods private.

Author: yhuai

sql/catalyst/src/main/scala/org/apache/spark/sql/types/dataTypes.scala

@@ -271,12 +271,12 @@ abstract class DataType {
   /** Check if `this` and `other` are the same data type when ignoring nullability
    *  (`StructField.nullable`, `ArrayType.containsNull`, and `MapType.valueContainsNull`).
    */
-  def sameType(other: DataType): Boolean = DataType.equalsIgnoreNullability(this, other)
+  private[sql] def sameType(other: DataType): Boolean = DataType.equalsIgnoreNullability(this, other)
 
   /** Returns the same data type but set all nullability fields are true
    * (`StructField.nullable`, `ArrayType.containsNull`, and `MapType.valueContainsNull`).
    */
-  def asNullable: DataType
+  private[sql] def asNullable: DataType
 }
 
 /**
@@ -293,7 +293,7 @@ class NullType private() extends DataType {
   // Defined with a private constructor so the companion object is the only possible instantiation.
   override def defaultSize: Int = 1
 
-  override def asNullable: NullType = this
+  private[sql] override def asNullable: NullType = this
 }
 
 case object NullType extends NullType
@@ -360,7 +360,7 @@ class StringType private() extends NativeType with PrimitiveType {
    */
   override def defaultSize: Int = 4096
 
-  override def asNullable: StringType = this
+  private[sql] override def asNullable: StringType = this
 }
 
 case object StringType extends StringType
@@ -396,7 +396,7 @@ class BinaryType private() extends NativeType with PrimitiveType {
    */
   override def defaultSize: Int = 4096
 
-  override def asNullable: BinaryType = this
+  private[sql] override def asNullable: BinaryType = this
 }
 
 case object BinaryType extends BinaryType
@@ -423,7 +423,7 @@ class BooleanType private() extends NativeType with PrimitiveType {
    */
   override def defaultSize: Int = 1
 
-  override def asNullable: BooleanType = this
+  private[sql] override def asNullable: BooleanType = this
 }
 
 case object BooleanType extends BooleanType
@@ -455,7 +455,7 @@ class TimestampType private() extends NativeType {
    */
   override def defaultSize: Int = 12
 
-  override def asNullable: TimestampType = this
+  private[sql] override def asNullable: TimestampType = this
 }
 
 case object TimestampType extends TimestampType
@@ -485,7 +485,7 @@ class DateType private() extends NativeType {
    */
   override def defaultSize: Int = 4
 
-  override def asNullable: DateType = this
+  private[sql] override def asNullable: DateType = this
 }
 
 case object DateType extends DateType
@@ -545,7 +545,7 @@ class LongType private() extends IntegralType {
 
   override def simpleString = "bigint"
 
-  override def asNullable: LongType = this
+  private[sql] override def asNullable: LongType = this
 }
 
 case object LongType extends LongType
@@ -576,7 +576,7 @@ class IntegerType private() extends IntegralType {
 
   override def simpleString = "int"
 
-  override def asNullable: IntegerType = this
+  private[sql] override def asNullable: IntegerType = this
 }
 
 case object IntegerType extends IntegerType
@@ -607,7 +607,7 @@ class ShortType private() extends IntegralType {
 
   override def simpleString = "smallint"
 
-  override def asNullable: ShortType = this
+  private[sql] override def asNullable: ShortType = this
 }
 
 case object ShortType extends ShortType
@@ -638,7 +638,7 @@ class ByteType private() extends IntegralType {
 
   override def simpleString = "tinyint"
 
-  override def asNullable: ByteType = this
+  private[sql] override def asNullable: ByteType = this
 }
 
 case object ByteType extends ByteType
@@ -706,7 +706,7 @@ case class DecimalType(precisionInfo: Option[PrecisionInfo]) extends FractionalT
     case None => "decimal(10,0)"
   }
 
-  override def asNullable: DecimalType = this
+  private[sql] override def asNullable: DecimalType = this
 }
 
 
@@ -766,7 +766,7 @@ class DoubleType private() extends FractionalType {
    */
   override def defaultSize: Int = 8
 
-  override def asNullable: DoubleType = this
+  private[sql] override def asNullable: DoubleType = this
 }
 
 case object DoubleType extends DoubleType
@@ -796,7 +796,7 @@ class FloatType private() extends FractionalType {
    */
   override def defaultSize: Int = 4
 
-  override def asNullable: FloatType = this
+  private[sql] override def asNullable: FloatType = this
 }
 
 case object FloatType extends FloatType
@@ -846,7 +846,7 @@ case class ArrayType(elementType: DataType, containsNull: Boolean) extends DataT
 
   override def simpleString = s"array<${elementType.simpleString}>"
 
-  override def asNullable: ArrayType = ArrayType(elementType.asNullable, containsNull = true)
+  private[sql] override def asNullable: ArrayType = ArrayType(elementType.asNullable, containsNull = true)
 }
 
 
@@ -1093,7 +1093,7 @@ case class StructType(fields: Array[StructField]) extends DataType with Seq[Stru
   private[sql] def merge(that: StructType): StructType =
     StructType.merge(this, that).asInstanceOf[StructType]
 
-  override def asNullable: StructType = {
+  private[sql] override def asNullable: StructType = {
     val newFields = fields.map {
       case StructField(name, dataType, nullable, metadata) =>
         StructField(name, dataType.asNullable, nullable = true, metadata)
@@ -1154,7 +1154,7 @@ case class MapType(
 
   override def simpleString = s"map<${keyType.simpleString},${valueType.simpleString}>"
 
-  override def asNullable: MapType =
+  private[sql] override def asNullable: MapType =
     MapType(keyType.asNullable, valueType.asNullable, valueContainsNull = true)
 }
 
@@ -1210,7 +1210,7 @@ abstract class UserDefinedType[UserType] extends DataType with Serializable {
    */
   override def defaultSize: Int = 4096
 
-  override def sameType(other: DataType): Boolean = ???
+  private[sql] override def sameType(other: DataType): Boolean = ???
 
-  override def asNullable: DataType = ???
+  private[sql] override def asNullable: DataType = ???
 }