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: all inspection queries hang during table dropping #13460

Closed
rjnn opened this issue Feb 7, 2017 · 4 comments
Closed

sql: all inspection queries hang during table dropping #13460

rjnn opened this issue Feb 7, 2017 · 4 comments
Assignees
Labels
A-sql-semantics C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior.
Milestone

Comments

@rjnn
Copy link
Contributor

rjnn commented Feb 7, 2017

I ran loadgen/ycsb workload B successfully for ~80 minutes, which ran at ~1800qps, and resulted in 514982 rows written (according to the load generator). For context, ycsb's schema looks like this:

CREATE TABLE IF NOT EXISTS usertable(ycsb_key INT PRIMARY KEY NOT NULL, FIELD1 TEXT, FIELD2 TEXT, FIELD3 TEXT,FIELD4 TEXT, FIELD5 TEXT, FIELD6 TEXT, FIELD7 TEXT, FIELD8 TEXT, FIELD9 TEXT, FIELD10 TEXT)

Dropping this table on a freshly restarted cluster is taking over 1.5 hours, with no way to see progress or status. While this is happening, all inspection queries fail, including SHOW TABLES or SHOW CREATE.... Of course, CREATE TABLE usertable also cannot complete until this finishes (#7348).

I believe this hanging occurs during all pending ALTER TABLE queries as well, but I have not reproduced a non-DROP version.

@rjnn
Copy link
Contributor Author

rjnn commented Feb 7, 2017

@petermattis could you please route this for me?

@petermattis
Copy link
Collaborator

@vivekmenezes DROP TABLE is considered as schema change operation, correct?

@petermattis petermattis changed the title All inspection queries hang during table dropping. sql: all inspection queries hang during table dropping Feb 23, 2017
@petermattis petermattis added this to the 1.0 milestone Feb 23, 2017
@knz knz added A-sql-semantics C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior. labels Feb 25, 2017
@vivekmenezes
Copy link
Contributor

Feel free to close this issue if you think #7348 will resolve it. The solution there is to rename the table before dropping it.

@vivekmenezes
Copy link
Contributor

#13908 resolves this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-sql-semantics C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior.
Projects
None yet
Development

No branches or pull requests

4 participants