GitHub user HyukjinKwon opened a pull request:

    https://github.com/apache/spark/pull/8275

    [SPARK-10035][SQL] Parquet filters does not process EqualNullSafe filter.

    As I talked with Lian, 
    
    1. I added EquelNullSafe to ParquetFilters
     - It uses the same equality comparison filter with EqualTo since the 
Parquet filter performs actually null-safe equality comparison.
    
    2. Updated the test code (ParquetFilterSuite) 
     - Convert catalyst.Expression to sources.Filter
     - Removed Cast since only Literal is picked up as a proper Filter in 
DataSourceStrategy
     - Added EquelNullSafe comparison
    
    3. Removed deprecated createFilter for catalyst.Expression


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/HyukjinKwon/spark master

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/spark/pull/8275.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #8275
    
----
commit 98ccf859fbfd7cce89bf56d625b0bd214638621a
Author: hyukjinkwon <gurwls...@gmail.com>
Date:   2015-08-18T12:41:00Z

    [SPARK-10035][SQL] Parquet filters does not process EqualNullSafe filter.

commit cf24b243eeb03eebb38ff14eeb9c61b9ead3178f
Author: hyukjinkwon <gurwls...@gmail.com>
Date:   2015-08-18T12:41:57Z

    [SPARK-10035][SQL] Update ParquetFilterSuite for sources.Filter test.

commit f35030f74f4bc0cf89fe815758f443a1912d1a95
Author: hyukjinkwon <gurwls...@gmail.com>
Date:   2015-08-18T12:46:42Z

    [SPARK-10035][SQL] Remove deprecated createFilter() in ParquetFilters (for 
catalyst.Expression).

----


---
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

Reply via email to