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

DROP OWNED BY fails to drop the schemas on the workers #5763

Closed
onderkalaci opened this issue Mar 4, 2022 · 4 comments · Fixed by #6519
Closed

DROP OWNED BY fails to drop the schemas on the workers #5763

onderkalaci opened this issue Mar 4, 2022 · 4 comments · Fixed by #6519
Labels

Comments

@onderkalaci
Copy link
Member

CREATE USER u1 WITH superuser;
SELECT run_command_on_workers($$CREATE USER u1 WITH superuser;$$);
\c - u1 - 
create schema test_sc;

DROP OWNED BY u1;

create schema test_sc;
ERROR:  schema "test_sc" already exists
CONTEXT:  while executing command on localhost:9700
@onderkalaci onderkalaci added the bug label Mar 4, 2022
TsinghuaLucky912 pushed a commit to TsinghuaLucky912/citus that referenced this issue Nov 24, 2022
TsinghuaLucky912 pushed a commit to TsinghuaLucky912/citus that referenced this issue Nov 27, 2022
@TsinghuaLucky912
Copy link
Contributor

#6519

TsinghuaLucky912 pushed a commit to TsinghuaLucky912/citus that referenced this issue Nov 29, 2022
marcocitus pushed a commit to TsinghuaLucky912/citus that referenced this issue Dec 2, 2022
marcocitus pushed a commit that referenced this issue Dec 2, 2022
Co-authored-by: TsinghuaLucky912 <tsinghualucky912@foxmail.com>
Fixes #5763
@onurctirtir
Copy link
Member

This doesn't seem to be fixed for all scenarios?:

create role non_super_user_role;
create schema my_schema;
alter schema my_schema owner to non_super_user_role ;
drop owned by non_super_user_role ;
create schema my_schema;
ERROR:  schema "my_schema" already exists
CONTEXT:  while executing command on localhost:9701

@marcocitus
Copy link
Member

marcocitus commented May 17, 2023

alter schema my_schema owner to non_super_user_role ;

seems like we're not propagating this one, so that's a different bug

@onurctirtir
Copy link
Member

onurctirtir commented May 17, 2023

oh okay, so I should add this into #4812

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants