Skip to content

Commit 947802c

Browse files
pzzsmarmbrus
authored andcommitted
[SPARK-6243][SQL] The Operation of match did not conside the scenarios that order.dataType does not match NativeType
It did not conside that order.dataType does not match NativeType. So i add "case other => ..." for other cenarios. Author: DoingDone9 <[email protected]> Closes apache#4959 from DoingDone9/case_ and squashes the following commits: 6278846 [DoingDone9] Update rows.scala cb1852d [DoingDone9] Merge pull request #2 from apache/master c3f046f [DoingDone9] Merge pull request #1 from apache/master
1 parent dfd2982 commit 947802c

File tree

1 file changed

+1
-0
lines changed
  • sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions

1 file changed

+1
-0
lines changed

sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/rows.scala

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -224,6 +224,7 @@ class RowOrdering(ordering: Seq[SortOrder]) extends Ordering[Row] {
224224
n.ordering.asInstanceOf[Ordering[Any]].compare(left, right)
225225
case n: NativeType if order.direction == Descending =>
226226
n.ordering.asInstanceOf[Ordering[Any]].reverse.compare(left, right)
227+
case other => sys.error(s"Type $other does not support ordered operations")
227228
}
228229
if (comparison != 0) return comparison
229230
}

0 commit comments

Comments
 (0)