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

Regression: Cancel Query function no longer works [CORE6458] #6691

Closed
firebird-automations opened this issue Dec 22, 2020 · 7 comments
Closed

Comments

@firebird-automations
Copy link
Collaborator

Submitted by: @artyom-smirnov

To reproduce it enough to create long query in isql and try to cancel it with Ctlr-C:

create database 'localhost:/tmp/test';
create sequence g;
select count(i) from (select gen_id(g,1) i from rdb$types a,rdb$types b,rdb$types c,rdb$types d);

Looks like it was broken after merging query restart functionality (9ac2238).

Commits: a5eb6ec

====== Test Details ======

Currently implemented for Windows only.

@firebird-automations
Copy link
Collaborator Author

Commented by: @pavel-zotov

Confirm.

WI-T4.0.0.1820 -- can be interrupted via Ctrl-C
WI-T4.0.0.1848 -- can not.

(SS used for both).

@firebird-automations
Copy link
Collaborator Author

Modified by: @dyemanov

assignee: Vlad Khorsun [ hvlad ]

@firebird-automations
Copy link
Collaborator Author

Modified by: Sean Leyne (seanleyne)

summary: Impossible to cancel query => Regression: Cancel Query function no longer works

@firebird-automations
Copy link
Collaborator Author

Commented by: @hvlad

Regression have limited scope - it happens only if
- query have resultset (for example SELECT is affected but EXECUTE PROCEDURE doesn't)
- query can't be canceled while first row is evaluated

@firebird-automations
Copy link
Collaborator Author

Modified by: @hvlad

status: Open [ 1 ] => Resolved [ 5 ]

resolution: Fixed [ 1 ]

Fix Version: 4.0 RC 1 [ 10930 ]

@firebird-automations
Copy link
Collaborator Author

Modified by: @pavel-zotov

status: Resolved [ 5 ] => Resolved [ 5 ]

QA Status: No test => Done successfully

Test Details: Currently implemented for Windows only.

Test Specifics: [Platform (Windows/Linux) specific]

@firebird-automations
Copy link
Collaborator Author

Modified by: @pavel-zotov

status: Resolved [ 5 ] => Closed [ 6 ]

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

No branches or pull requests

2 participants