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

Add test for DROP TABLE IF EXISTS statement with non-existing object #15464

Merged

Conversation

krvikash
Copy link
Contributor

@krvikash krvikash commented Dec 19, 2022

Description

Add test for DROP TABLE IF EXISTS statement with non-existing object

Additional context and related issues

NA

Release notes

(X) This is not user-visible or docs only and no release notes are required.
( ) Release notes are required, please propose a release note for me.
( ) Release notes are required, with the following suggested text:

# Section
* Fix some things. ({issue}`issuenumber`)

@cla-bot cla-bot bot added the cla-signed label Dec 19, 2022
@krvikash krvikash added the no-release-notes This pull request does not require release notes entry label Dec 19, 2022
@findepi findepi added the test label Dec 19, 2022
@findepi
Copy link
Member

findepi commented Dec 19, 2022

@krvikash there was a force push to the master branch (per #15365 (comment)), so your PR appears as if containing some unrelated commits.
Can you please rebase to remove them?

@krvikash krvikash force-pushed the add-tests-for-drop-table-if-exists-statement branch 4 times, most recently from bb31cd2 to d10fd45 Compare December 21, 2022 19:58
@krvikash krvikash force-pushed the add-tests-for-drop-table-if-exists-statement branch from d10fd45 to 470843c Compare December 22, 2022 06:48
@krvikash krvikash self-assigned this Dec 22, 2022
@ebyhr
Copy link
Member

ebyhr commented Dec 22, 2022

CI hit #15367

@ebyhr ebyhr merged commit 652664c into trinodb:master Dec 22, 2022
@github-actions github-actions bot added this to the 404 milestone Dec 22, 2022
@krvikash krvikash deleted the add-tests-for-drop-table-if-exists-statement branch December 22, 2022 08:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed no-release-notes This pull request does not require release notes entry test
Development

Successfully merging this pull request may close these issues.

4 participants