GitHub user ericl opened a pull request:

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

    [SPARK-17974] Refactor FileCatalog classes to simplify the inheritance tree

    ## What changes were proposed in this pull request?
    
    This renames `BasicFileCatalog => FileCatalog`, combines  
`SessionFileCatalog` with `PartitioningAwareFileCatalog`, and removes the old 
`FileCatalog` trait.
    
    In summary,
    ```
    MetadataLogFileCatalog extends PartitioningAwareFileCatalog
    ListingFileCatalog extends PartitioningAwareFileCatalog
    PartitioningAwareFileCatalog extends FileCatalog
    TableFileCatalog extends FileCatalog
    ```
    
    cc @cloud-fan @mallman 
    
    ## How was this patch tested?
    
    Existing tests

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

    $ git pull https://github.com/ericl/spark refactor-session-file-catalog

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

    https://github.com/apache/spark/pull/15518.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 #15518
    
----
commit 798a8820c074e17b1eb17ca4446c7eb0969c63c8
Author: Eric Liang <e...@databricks.com>
Date:   2016-10-17T18:30:30Z

    move files

commit db8a669bc1c51fef97e231cc5fec4eb9523b9c47
Author: Eric Liang <e...@databricks.com>
Date:   2016-10-17T18:31:42Z

    Mon Oct 17 11:31:42 PDT 2016

commit e2d8c1d850bfaa30dd4b598d284f6068044c46cd
Author: Eric Liang <e...@databricks.com>
Date:   2016-10-17T19:05:47Z

    clean up classes

commit 5d390c97f61af4d84c194344d8dd4c86fb41eb32
Author: Eric Liang <e...@databricks.com>
Date:   2016-10-17T19:08:28Z

    merge suites

----


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