Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[SPARK-23000] Use fully qualified table names in HiveMetastoreCatalogSuite #20273

Closed
wants to merge 1 commit into from
Closed

[SPARK-23000] Use fully qualified table names in HiveMetastoreCatalogSuite #20273

wants to merge 1 commit into from

Conversation

sameeragarwal
Copy link
Member

What changes were proposed in this pull request?

In another attempt to fix DataSourceWithHiveMetastoreCatalogSuite, this patch uses qualified table names (default.t) in the individual tests.

How was this patch tested?

N/A (Test Only Change)

@sameeragarwal
Copy link
Member Author

cc @gatorsmile

@SparkQA
Copy link

SparkQA commented Jan 16, 2018

Test build #86145 has finished for PR 20273 at commit 3de8b41.

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

gatorsmile
gatorsmile reviewed Jan 16, 2018
View reviewed changes
Copy link
Member

@gatorsmile gatorsmile left a comment

Choose a reason for hiding this comment

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

LGTM

Let us see whether it can make the test pass. :)

@gatorsmile
Copy link
Member

Thanks! Merged to master/2.3

asfgit pushed a commit that referenced this pull request Jan 16, 2018
@sameeragarwal @gatorsmile
[SPARK-23000] Use fully qualified table names in HiveMetastoreCatalog…
e2ffb97 
…Suite

## What changes were proposed in this pull request?

In another attempt to fix DataSourceWithHiveMetastoreCatalogSuite, this patch uses qualified table names (`default.t`) in the individual tests.

## How was this patch tested?

N/A (Test Only Change)

Author: Sameer Agarwal <[email protected]>

Closes #20273 from sameeragarwal/flaky-test.

(cherry picked from commit c7572b7)
Signed-off-by: gatorsmile <[email protected]>
@asfgit asfgit closed this in c7572b7 Jan 16, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gatorsmile gatorsmile gatorsmile left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sameeragarwal @SparkQA @gatorsmile