Skip to content
Closed
Changes from 1 commit
Commits
Show all changes
22 commits
Select commit Hold shift + click to select a range
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Simplify isSearchableType
  • Loading branch information
dongjoon-hyun committed Nov 29, 2017
commit 6971cdf7d0330b933c09d11fe415f3aada10609d
Original file line number Diff line number Diff line change
Expand Up @@ -50,11 +50,12 @@ private[orc] object OrcFilters {

/**
* Return true if this is a searchable type in ORC.
* Both CharType and VarcharType are cleaned at AstBuilder.
*/
private def isSearchableType(dataType: DataType) = dataType match {
case ByteType | ShortType | FloatType | DoubleType => true
case IntegerType | LongType | StringType | BooleanType => true
case TimestampType | _: DecimalType => true
// TODO: SPARK-21787 Support for pushing down filters for DateType in ORC
case BinaryType | DateType => false
case _: AtomicType => true
case _ => false
}

Expand Down