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.

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

[SPARK-32405][SQL][FOLLOWUP] Remove USING _ in CREATE TABLE in JDBCTableCatalog docker tests #30599

Closed
wants to merge 1 commit into from

Conversation

huaxingao
Copy link
Contributor

What changes were proposed in this pull request?

remove USING _ in CREATE TABLE in JDBCTableCatalog docker tests

Why are the changes needed?

Previously CREATE TABLE syntax forces users to specify a provider so we have to add a USING _ . Now the problem was fix and we need to remove it.

Does this PR introduce any user-facing change?

No

How was this patch tested?

Existing tests

@github-actions github-actions bot added the SQL label Dec 4, 2020
@SparkQA
Copy link

SparkQA commented Dec 4, 2020

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/36777/

@SparkQA
Copy link

SparkQA commented Dec 4, 2020

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/36777/

@huaxingao
Copy link
Contributor Author

@cloud-fan Sorry, forgot to remove these last time.

@SparkQA
Copy link

SparkQA commented Dec 4, 2020

Test build #132176 has finished for PR 30599 at commit c8bbbce.

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

@cloud-fan
Copy link
Contributor

thanks, merging to master!

@cloud-fan cloud-fan closed this in e22ddb6 Dec 4, 2020
@huaxingao
Copy link
Contributor Author

Thanks!

@huaxingao huaxingao deleted the remove_USING branch December 4, 2020 06:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
3 participants