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

sql: DROP DATABASE CASCADE not working in the 3node-tenant configuration #50840

Closed
arulajmani opened this issue Jun 30, 2020 · 2 comments
Closed
Labels
A-multitenancy Related to multi-tenancy C-investigation Further steps needed to qualify. C-label will change.

Comments

@arulajmani
Copy link
Collaborator

While trying to add a logictest, I discovered the following doesn't work under the 3node-tenant configuration:

statement
CREATE DATABASE db

statement ok
CREATE TABLE db.t(a int)

statement ok
DROP DATABASE db CASCADE

cc @asubiotto @nvanbenschoten

@blathers-crl
Copy link

blathers-crl bot commented Jun 30, 2020

Hi @arulajmani, please add a C-ategory label to your issue. Check out the label system docs.

While you're here, please consider adding an A- label to help keep our repository tidy.

🦉 Hoot! I am a Blathers, a bot for CockroachDB. My owner is otan.

@arulajmani arulajmani added A-multitenancy Related to multi-tenancy C-investigation Further steps needed to qualify. C-label will change. labels Jun 30, 2020
@asubiotto
Copy link
Contributor

I think this is the same issue as #50048. Feel free to blacklist that config and point to that issue (i.e. # LogicTest: !3node-tenant(50048)

arulajmani added a commit to arulajmani/cockroach that referenced this issue Jul 7, 2020
Logictests for sequences were earlier blocked on issue cockroachdb#50840. From the
discussion on the issue, the root cause is cockroachdb#50048. This PR updates
the logictest blocklist to reflect that.

Release note: none
craig bot pushed a commit that referenced this issue Jul 9, 2020
51062: sql: update logictest blocklist issue number r=asubiotto a=arulajmani

Logictests for sequences were earlier blocked on issue #50840. From the
discussion on the issue, the root cause is #50048. This PR updates
the logictest blocklist to reflect that.

Release note: none

Co-authored-by: arulajmani <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-multitenancy Related to multi-tenancy C-investigation Further steps needed to qualify. C-label will change.
Projects
None yet
Development

No branches or pull requests

2 participants