Github user rxin commented on a diff in the pull request:

    https://github.com/apache/spark/pull/10689#discussion_r49292062
  
    --- Diff: 
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/CatalystQlSuite.scala 
---
    @@ -49,4 +50,16 @@ class CatalystQlSuite extends PlanTest {
         parser.createPlan("select sum(product + 1) over (partition by (product 
+ (1)) order by 2) " +
           "from windowData")
       }
    +
    +  test("limit clause: a support in set operation") {
    +    val plan1 = parser.createPlan("select key from (select * from t1) x 
limit 1")
    +    assert(plan1.collect{ case w: Limit => w }.size === 1)
    +
    +    val plan2 = parser.createPlan("select key from (select * from t1 limit 
2) x limit 1")
    +    assert(plan2.collect{ case w: Limit => w }.size === 2)
    +
    +    val plan3 = parser.createPlan("select key from ((select * from 
testData limit 1) " +
    +      "union all (select * from testData limit 1)) x limit 1")
    +    assert(plan3.collect{ case w: Limit => w }.size === 3)
    --- End diff --
    
    can we do this similar to how we compare plans in various optimizer suites, 
e.g. 
https://github.com/apache/spark/blob/master/sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/OptimizeInSuite.scala#L84


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

Reply via email to