Skip to content

Commit

Permalink
Re-enable NullType sorting test now that SPARK-8868 is fixed
Browse files Browse the repository at this point in the history
  • Loading branch information
JoshRosen committed Jul 8, 2015
1 parent 844f4ca commit d31f180
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ class UnsafeExternalSortSuite extends SparkPlanTest with BeforeAndAfterAll {

// Test sorting on different data types
for (
dataType <- DataTypeTestUtils.atomicTypes // Disable null type for now due to bug in SqlSerializer2 ++ Set(NullType);
dataType <- DataTypeTestUtils.atomicTypes ++ Set(NullType)
if !dataType.isInstanceOf[DecimalType]; // Since we don't have an unsafe representation for decimals
nullable <- Seq(true, false);
sortOrder <- Seq('a.asc :: Nil, 'a.desc :: Nil);
Expand Down

0 comments on commit d31f180

Please sign in to comment.