Github user srowen commented on the issue:
https://github.com/apache/spark/pull/15925
That isn't the same case. In one instance you are showing an example of SQL
syntax, and in the other, usage of a programmatic Scala API. The SQL query is
not semantically wrong, but syntactically wrong.
It would at least be OK to make sure that the "IN ()" case is generated in
a way that works for all SQL engines. If it's always correct to write "IN ('')"
then that's fine, but, doesn't that mean "a set containing the empty string" in
some cases? I'm sort of surprised that's the way to express it in some SQL
engines. Is it really empty?
---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
