HyukjinKwon commented on a change in pull request #29886:
URL: https://github.com/apache/spark/pull/29886#discussion_r495677916
##########
File path:
sql/core/src/test/resources/sql-tests/results/postgreSQL/window_part3.sql.out
##########
@@ -391,7 +391,7 @@ SELECT nth_value(four, 0) OVER (ORDER BY ten), ten, four
FROM tenk1
struct<>
-- !query output
org.apache.spark.sql.AnalysisException
-cannot resolve 'nthvalue(spark_catalog.default.tenk1.`four`, 0)' due to data
type mismatch: The 'offset' argument of nth_value must be greater than zero but
it is 0.; line 1 pos 7
+cannot resolve 'nth_value(spark_catalog.default.tenk1.`four`, 0)()' due to
data type mismatch: The 'offset' argument of nth_value must be greater than
zero but it is 0.; line 1 pos 7
Review comment:
Do you know why we have an extra `()`?
----------------------------------------------------------------
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:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
