Skip to content

[SPARK-18103][Follow-up][SQL][Minor] Rename MetadataLogFileCatalog to MetadataLogFileIndex - #15712

Closed
lw-lin wants to merge 1 commit into
apache:masterfrom
lw-lin:18103
Closed

[SPARK-18103][Follow-up][SQL][Minor] Rename MetadataLogFileCatalog to MetadataLogFileIndex#15712
lw-lin wants to merge 1 commit into
apache:masterfrom
lw-lin:18103

Conversation

@lw-lin

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

This is a follow-up to #15634.

How was this patch tested?

N/A

@lw-lin

lw-lin commented Nov 1, 2016

Copy link
Copy Markdown
ContributorAuthor

@rxin@ericl could you take a look thanks!

@lw-linlw-lin changed the title [SPARK-18103][SQL] Rename MetadataLogFileCatalog to MetadataLogFileIndex[SPARK-18103][Follow-up][SQL] Rename MetadataLogFileCatalog to MetadataLogFileIndexNov 1, 2016
@SparkQA

Copy link
Copy Markdown

Test build #67884 has finished for PR 15712 at commit d5680b3.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@lw-linlw-lin changed the title [SPARK-18103][Follow-up][SQL] Rename MetadataLogFileCatalog to MetadataLogFileIndex[SPARK-18103][Follow-up][SQL][Minor] Rename MetadataLogFileCatalog to MetadataLogFileIndexNov 1, 2016

@ericlericl left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for catching this

@rxin

rxin commented Nov 1, 2016

Copy link
Copy Markdown
Contributor

Thanks - merging in master.

@asfgitasfgit closed this in 8cdf143Nov 1, 2016
@lw-lin
lw-lin deleted the 18103 branch November 2, 2016 03:53
uzadude pushed a commit to uzadude/spark that referenced this pull request Jan 27, 2017
…to `MetadataLogFileIndex`
## What changes were proposed in this pull request?
This is a follow-up to apache#15634.
## How was this patch tested?
N/A
Author: Liwei Lin <lwlin7@gmail.com>
Closesapache#15712 from lw-lin/18103.
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@lw-lin@SparkQA@rxin@ericl