younggyuchun edited a comment on issue #25195: 
[SPARK-28288][SQL][PYTHON][TESTS] Convert and port 'window.sql' into …
URL: https://github.com/apache/spark/pull/25195#issuecomment-513079590
 
 
   @HyukjinKwon The reason is that I specified the "ORDER BY val"?:
   
   Before:
   `SELECT val, cate, row_number() OVER(PARTITION BY cate) FROM testData ORDER 
BY cate, val;`
   
   After:
   `SELECT udf(val), cate, row_number() OVER(PARTITION BY cate ORDER BY val) 
FROM testData ORDER BY cate, udf(val)`
   
   I guess we shouldn't specify "ORDER BY val" here right?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

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

Reply via email to