Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/15704#discussion_r86393914
--- Diff:
sql/catalyst/src/main/antlr4/org/apache/spark/sql/catalyst/parser/SqlBase.g4 ---
@@ -243,7 +243,7 @@ partitionSpec
;
partitionVal
- : identifier (EQ constant)?
+ : identifier (comparisonOperator constant)?
--- End diff --
@dongjoon-hyun that is an interesting point. We might need to take a step
back here. The only thing I want to prevent from happening is that we are going
to recreate an expression hierarchy, and not reuse what is already there. That
being said, things must remain concise. So if it takes a large amount of time
or code, then we should reconsider.
In this case I would like to try the expression approach first if you don't
mind.
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
