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

IGNITE-23939 Query cancellation shuld wait for the query termination phase only #4945

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

xtern
Copy link
Contributor

@xtern xtern commented Dec 20, 2024

https://issues.apache.org/jira/browse/IGNITE-23939

To achieve consistent behavior between query cancellation and script cancellation, query cancellation should only wait the termination phase and don't wait until the query is removed from the running queries registry.

Note: fixed tests to conditionally wait until the query is removed from the registry.

p.s. the main change have been made in Query and QueryExecutor classes.

@xtern xtern changed the title IGNITE-23939 Simplify query cancellation. IGNITE-23939 Canceling a query should not wait until the query is removed from the running queries registry Dec 20, 2024
@xtern xtern changed the title IGNITE-23939 Canceling a query should not wait until the query is removed from the running queries registry IGNITE-23939 Query cancellation should not wait until the query is removed from the running queries registry Dec 20, 2024
@xtern xtern changed the title IGNITE-23939 Query cancellation should not wait until the query is removed from the running queries registry IGNITE-23939 Query cancellation shuld wait for the query termination phase only Dec 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant