Skip to content

[SPARK-21699][SQL] Remove unused getTableOption in ExternalCatalog - #18912

Closed
rxin wants to merge 1 commit into
apache:masterfrom
rxin:remove-getTableOption
Closed

[SPARK-21699][SQL] Remove unused getTableOption in ExternalCatalog#18912
rxin wants to merge 1 commit into
apache:masterfrom
rxin:remove-getTableOption

Conversation

@rxin

@rxinrxin commented Aug 10, 2017

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

This patch removes the unused SessionCatalog.getTableMetadataOption and ExternalCatalog. getTableOption.

How was this patch tested?

Removed the test case.

@gatorsmile

Copy link
Copy Markdown
Member

LGTM except the PR title

@rxinrxin changed the title [SQL] Remove unused getTableOption in ExternalCatalog[SPARK-21699][SQL] Remove unused getTableOption in ExternalCatalogAug 10, 2017
@SparkQA

Copy link
Copy Markdown

Test build #80510 has finished for PR 18912 at commit 5be8011.

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

@rxin

rxin commented Aug 11, 2017

Copy link
Copy Markdown
ContributorAuthor

Merging in master / 2.2.

asfgit pushed a commit that referenced this pull request Aug 11, 2017
## What changes were proposed in this pull request?
This patch removes the unused SessionCatalog.getTableMetadataOption and ExternalCatalog. getTableOption.
## How was this patch tested?
Removed the test case.
Author: Reynold Xin <rxin@databricks.com>
Closes#18912 from rxin/remove-getTableOption.
(cherry picked from commit 584c7f1)
Signed-off-by: Reynold Xin <rxin@databricks.com>
MatthewRBruce pushed a commit to Shopify/spark that referenced this pull request Jul 31, 2018
## What changes were proposed in this pull request?
This patch removes the unused SessionCatalog.getTableMetadataOption and ExternalCatalog. getTableOption.
## How was this patch tested?
Removed the test case.
Author: Reynold Xin <rxin@databricks.com>
Closesapache#18912 from rxin/remove-getTableOption.
(cherry picked from commit 584c7f1)
Signed-off-by: Reynold Xin <rxin@databricks.com>
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.

3 participants

@rxin@gatorsmile@SparkQA