cloud-fan commented on a change in pull request #26285: [SPARK-29623][SQL] do
not allow multiple unit TO unit statements in interval literal syntax
URL: https://github.com/apache/spark/pull/26285#discussion_r340542625
##########
File path:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/parser/ExpressionParserSuite.scala
##########
@@ -667,17 +667,13 @@ class ExpressionParserSuite extends AnalysisTest {
}
// Unknown FROM TO intervals
- intercept("interval 10 month to second",
+ intercept("interval '10' month to second",
Review comment:
If you do write `interval 10 month to second`, then the parser thinks
`interval 10 month to` is a valid expression (interva literal with alias) and
`second` should be removed.
----------------------------------------------------------------
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]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
