MaxGekk commented on a change in pull request #32409:
URL: https://github.com/apache/spark/pull/32409#discussion_r629088422
##########
File path:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/CollectionExpressionsSuite.scala
##########
@@ -1098,7 +1098,7 @@ class CollectionExpressionsSuite extends SparkFunSuite
with ExpressionEvalHelper
Literal(Date.valueOf("2018-01-05")),
Literal(Period.ofDays(2))),
EmptyRow,
- "sequence step must be a day year-month interval if start and end
values are dates")
+ "sequence step must be a day interval year to month if start and end
values are dates")
Review comment:
@beliefer The error message confuses me slightly, especially the
combination `a day interval year to month`. Could you open a PR to improve the
error, please, something like "... sequence step must be an interval of day
granularity ...".
--
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]
