Github user cloud-fan commented on a diff in the pull request:

    https://github.com/apache/spark/pull/21291#discussion_r187900810
  
    --- Diff: 
sql/core/src/test/scala/org/apache/spark/sql/execution/PlannerSuite.scala ---
    @@ -621,6 +621,25 @@ class PlannerSuite extends SharedSQLContext {
           requiredOrdering = Seq(orderingA, orderingB),
           shouldHaveSort = true)
       }
    +
    +  test("SPARK-24242: RangeExec should have correct output ordering") {
    +    val df = spark.range(10).orderBy("id")
    --- End diff --
    
    why do we put an `orderBy` in the query?


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to