diff options
author | Reynold Xin <rxin@databricks.com> | 2016-07-06 15:04:37 -0700 |
---|---|---|
committer | Reynold Xin <rxin@databricks.com> | 2016-07-06 15:04:37 -0700 |
commit | 8e3e4ed6c090d18675d49eec46b3ee572457db95 (patch) | |
tree | 4c3adc90097f9f12eb43346b6169befe0dcaf2b0 /sql/catalyst/src/main | |
parent | 9c041990cf4d0138d9104207b5c2e7a319b42615 (diff) | |
download | spark-8e3e4ed6c090d18675d49eec46b3ee572457db95.tar.gz spark-8e3e4ed6c090d18675d49eec46b3ee572457db95.tar.bz2 spark-8e3e4ed6c090d18675d49eec46b3ee572457db95.zip |
[SPARK-16371][SQL] Two follow-up tasks
## What changes were proposed in this pull request?
This is a small follow-up for SPARK-16371:
1. Hide removeMetadata from public API.
2. Add JIRA ticket number to test case name.
## How was this patch tested?
Updated a test comment.
Author: Reynold Xin <rxin@databricks.com>
Closes #14074 from rxin/parquet-filter.
Diffstat (limited to 'sql/catalyst/src/main')
-rw-r--r-- | sql/catalyst/src/main/scala/org/apache/spark/sql/types/StructType.scala | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sql/catalyst/src/main/scala/org/apache/spark/sql/types/StructType.scala b/sql/catalyst/src/main/scala/org/apache/spark/sql/types/StructType.scala index 0e89f71dc1..0284ecc0d9 100644 --- a/sql/catalyst/src/main/scala/org/apache/spark/sql/types/StructType.scala +++ b/sql/catalyst/src/main/scala/org/apache/spark/sql/types/StructType.scala @@ -413,10 +413,10 @@ object StructType extends AbstractDataType { StructType(fields.asScala) } - protected[sql] def fromAttributes(attributes: Seq[Attribute]): StructType = + private[sql] def fromAttributes(attributes: Seq[Attribute]): StructType = StructType(attributes.map(a => StructField(a.name, a.dataType, a.nullable, a.metadata))) - def removeMetadata(key: String, dt: DataType): DataType = + private[sql] def removeMetadata(key: String, dt: DataType): DataType = dt match { case StructType(fields) => val newFields = fields.map { f => |