Github user liancheng commented on the pull request:
https://github.com/apache/spark/pull/3157#issuecomment-67770452
@srowen Sorry for the late reply, missed this thread. Yes, the
`Operators.Not` should be replaced with `Operators.NotEq`. The original Parquet
filter test cases in `ParquetQuerySuite` didn't catch this error because the
type information in [this
assertion](https://github.com/apache/spark/blob/master/sql/core/src/test/scala/org/apache/spark/sql/parquet/ParquetQuerySuite.scala#L466)
is actually erased at runtime.
On the other hand, `ParquetQuerySuite` will be removed soon. It has been
deprecated by a new set of Parquet test suites introduced in #3644. Similar
type erasure problem doesn't exist in the new `ParquetFilterSuite`.
---
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]
