address comment

mgaido91 · mgaido91 · commit 6c5f04cb9897 · 2018-04-13T11:50:58.000+02:00
diff --git a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
@@ -740,8 +740,7 @@ object EliminateSorts extends Rule[LogicalPlan] {
  */
 object RemoveRedundantSorts extends Rule[LogicalPlan] {
   def apply(plan: LogicalPlan): LogicalPlan = plan transform {
-    case Sort(orders, true, child) if child.outputOrdering.nonEmpty
-        && SortOrder.orderingSatisfies(child.outputOrdering, orders) =>
+    case Sort(orders, true, child) if SortOrder.orderingSatisfies(child.outputOrdering, orders) =>
       child
   }
 }

Original file line number	Diff line number	Diff line change
`@@ -740,8 +740,7 @@ object EliminateSorts extends Rule[LogicalPlan] {`
`740`	`740`	`*/`
`741`	`741`	`object RemoveRedundantSorts extends Rule[LogicalPlan] {`
`742`	`742`	`def apply(plan: LogicalPlan): LogicalPlan = plan transform {`
`743`		`- case Sort(orders, true, child) if child.outputOrdering.nonEmpty`
`744`		`- && SortOrder.orderingSatisfies(child.outputOrdering, orders) =>`
	`743`	`+ case Sort(orders, true, child) if SortOrder.orderingSatisfies(child.outputOrdering, orders) =>`
`745`	`744`	`child`
`746`	`745`	`}`
`747`	`746`	`}`