Files
impala/tests/util/cancel_util.py
Michael Smith ec6585fa7e IMPALA-915: Support cancel queries in frontend
Adds support to cancel a query during Frontend planning or metadata
operations. Frontend planning is handled by createExecRequest, so
registers Java Threads executing createExecRequest by their query ID and
provides cancelExecRequest to interrupt the Thread for a particular
query ID.

Cancellation is implemented by setting a boolean for the thread, and
calling Thread.interrupt to trigger InterruptedException from any wait
calls. Several ignored wait calls are updated to check the boolean and
throw an exception if the query has been cancelled, interrupting those
operations.

Adds periodic checks to the planning process to interrupt planning.
They're primarily useful when planning is waiting on catalogd/HMS. If
planning gets into an algorithmically complex operation, it will not be
interrupted.

Removes check_inflight, as we can now cancel a query before it's
inflight. In the case that cancellation doesn't happen immediately -
because we're in a busy frontend loop that can't be interrupted -
/cancel will block until the frontend reaches an interruption point and
returns to the backend to finalize the query.

When analysis returns, cancellation is finalized in the backend. The
/cancel_query request returns once the query is cancelled. Cancelling
a request can no longer fail, so additional checks for whether the
request has been cancelled before it started executing are added.

Removes setting UpdateQueryStatus when GetExecRequest returns because
that's already handled in ImpalaServer::Execute when it calls
UnregisterQuery in response to an error, and constitutes an update race
on the status with UnregisterQuery triggered by CancelQueryHandler. We
want to use the status from CancelQueryHandler in this case as it
provides more context (about who initiated the cancel); the result of
GetExecRequest is just UserCancelledException. Avoids calling
UnregisterQuery in Execute if the query is already finalized to avoid
redundant "Invalid or unknown query handle" logs.

Extends idle_query_statuses_ to save status for any query interrupted by
an external process - cancelled by a user or timeout - so they can be
handled consistently.

Testing:
- updates test_query_cancel_created to cancel a CREATED query
- added tests to cancel a query while metadata loading is delayed
- removes test_query_cancel_exception, as it no longer demonstrates
  relevant behavior; cancelling a query that will encounter an exception
  before the exception occurs is no different than other queries
- ran query_test/test_cancellation.py in exhaustive mode
- ran query_test/test_cancellation.py w/ DEFAULT_TEST_PROTOCOL=beeswax
- updates cancellation tests that expect INVALID_QUERY_HANDLE to accept
  Cancelled, which is sometimes returned by interrupted query status.

Change-Id: I0d25d4c7fb0b8dcc7dad9510db1e8dca220eeb86
Reviewed-on: http://gerrit.cloudera.org:8080/21803
Reviewed-by: Impala Public Jenkins <impala-public-jenkins@cloudera.com>
Tested-by: Impala Public Jenkins <impala-public-jenkins@cloudera.com>
2025-07-11 22:45:13 +00:00

275 lines
11 KiB
Python

# Licensed to the Apache Software Foundation (ASF) under one
# or more contributor license agreements. See the NOTICE file
# distributed with this work for additional information
# regarding copyright ownership. The ASF licenses this file
# to you under the Apache License, Version 2.0 (the
# "License"); you may not use this file except in compliance
# with the License. You may obtain a copy of the License at
#
# http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing,
# software distributed under the License is distributed on an
# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
# KIND, either express or implied. See the License for the
# specific language governing permissions and limitations
# under the License.
from __future__ import absolute_import, division, print_function
import threading
from time import sleep
import traceback
from impala_thrift_gen.TCLIService import TCLIService
from tests.common.impala_connection import (
create_connection,
ERROR,
IMPALA_CONNECTION_EXCEPTION,
MinimalHS2Connection,
)
from tests.common.impala_test_suite import IMPALAD_HS2_HOST_PORT, ImpalaTestSuite
from tests.common.test_result_verifier import error_msg_startswith
class QueryToKill:
def __init__(self, test_suite, protocol, check_on_exit=True, user=None, nth_impalad=0):
self.client = test_suite.create_client_for_nth_impalad(nth_impalad, protocol)
self.sql = 'SELECT sleep(1000)'
self.check_on_exit = check_on_exit
self.user = user
def poll(self):
while True:
try:
results = self.client.fetch(self.sql, self.handle)
if len(results.data) > 0:
raise Exception("Failed to kill query within time limit.")
except Exception as e:
self.exc = e
return
def __enter__(self):
self.handle = self.client.execute_async(self.sql, user=self.user)
self.poll_thread = threading.Thread(target=lambda: self.poll())
self.poll_thread.start()
return self.client.handle_id(self.handle)
def __exit__(self, exc_type, exc_value, traceback): # noqa: U100
self.poll_thread.join()
if not self.check_on_exit:
self.client.close()
return
# If ImpalaServer::UnregisterQuery() happens before the last polling, the error
# message will be "Invalid or unknown query handle". Otherwise, the error message
# will be "Cancelled".
assert error_msg_startswith(
str(self.exc),
["Invalid or unknown query handle", "Cancelled"],
self.client.handle_id(self.handle),
)
try:
self.client.fetch(self.sql, self.handle)
except Exception as ex:
assert "Invalid or unknown query handle" in str(ex) or "Cancelled" in str(ex)
finally:
self.client.close()
def assert_kill_ok(client, query_id, user=None):
sql = "KILL QUERY '{0}'".format(query_id)
result = client.execute(sql, user=user)
assert result.success and len(result.data) == 1
assert result.data[0] == "Query {0} is killed.".format(query_id)
def assert_kill_error(client, error_msg, query_id=None, sql=None, user=None):
if sql is None:
sql = "KILL QUERY '{0}'".format(query_id)
try:
client.execute(sql, user=user)
assert False, "Failed to catch the exception."
except Exception as exc:
assert error_msg_startswith(str(exc), error_msg)
class FetchingThread(threading.Thread):
"""Thread that does rows fetching that is subject to cancellation from main thread.
execute_async() must be called before starting the thread.
"""
def __init__(self, client, query, exec_option, table_format,
use_kill_query_statement=False):
super(FetchingThread, self).__init__(name='FetchingThread')
self.client = client
self.query = query
self.exec_option = exec_option
self.table_format = table_format
self.db_name = ImpalaTestSuite.get_db_name_from_format(table_format)
self.use_kill_query_statement = use_kill_query_statement
self.fetch_results_error = None
self.query_profile = None
self.handle = None
self.is_error = False
def execute_async(self):
# change database.
use_handle = self.client.execute_async('use ' + self.db_name)
self.client.wait_for(use_handle)
self.client.close_query(use_handle)
self.client.set_configuration(self.exec_option)
# execute the query.
self.handle = self.client.execute_async(self.query)
self.is_error = self.client.get_impala_exec_state(self.handle) == ERROR
def run(self):
if self.is_error:
self.client.log_client(
'Not starting fetch_results because query failed to start')
return
self.client.log_client('Start fetching results')
try:
result = True
while result:
result = self.client.fetch(self.query, self.handle)
except IMPALA_CONNECTION_EXCEPTION as e:
self.fetch_results_error = e
except Exception as e:
stack_trace_string = traceback.format_exc()
msg = "Exception in fetch_results: {}\n{}".format(
str(e), stack_trace_string)
self.fetch_results_error = Exception(msg)
finally:
self.client.log_client('Stop fetching results')
def cancel_query(self):
assert self.handle is not None, \
"handle is None. Did the query fail to start?"
if self.use_kill_query_statement:
# Run kill query using ImpylaHS2Connection.
with create_connection(
host_port=self.client.get_host_port(),
protocol=self.client.get_test_protocol(),
) as kill_client:
kill_client.connect()
if self.exec_option:
kill_client.set_configuration(self.exec_option)
assert_kill_ok(kill_client, self.client.handle_id(self.handle))
else:
# Run cancellation using separate client/connection.
with MinimalHS2Connection(IMPALAD_HS2_HOST_PORT) as cancel_client:
cancel_resp = cancel_client.cancel(self.handle)
assert cancel_resp.status.statusCode == TCLIService.TStatusCode.SUCCESS_STATUS
def close_query(self):
with MinimalHS2Connection(IMPALAD_HS2_HOST_PORT) as close_client:
close_client.close_query(self.handle)
def get_runtime_profile(self):
"""Get query profile.
Might hit exception if QueryState has been released."""
try:
return self.client.get_runtime_profile(self.handle)
except Exception:
return None
def cancel_query_and_validate_state(
query, exec_option, table_format, cancel_delay, join_before_close=False,
use_kill_query_statement=False):
"""Runs the given query asynchronously and then cancels it after the specified delay.
The query is run with the given 'exec_options' against the specified 'table_format'. A
separate async thread is launched to fetch the results of the query. The method
validates that the query was successfully cancelled and that the error messages for the
calls to ImpalaConnection#fetch and #close are consistent. If 'join_before_close' is
True the method will join against the fetch results thread before closing the query.
If 'use_kill_query_statement' is True and 'join_before_close' is False, a KILL QUERY
statement will be executed to cancel and close the query, instead of sending the Thrift
RPCs directly.
"""
assert table_format is not None
assert not (join_before_close and use_kill_query_statement)
with MinimalHS2Connection(IMPALAD_HS2_HOST_PORT) as fetch_client:
thread = FetchingThread(fetch_client, query, exec_option, table_format,
use_kill_query_statement=use_kill_query_statement)
__run_cancel_query_and_validate_state(
thread, cancel_delay, join_before_close, use_kill_query_statement)
def __run_cancel_query_and_validate_state(
thread, cancel_delay, join_before_close=False, use_kill_query_statement=False):
thread.execute_async()
thread.start()
sleep(cancel_delay)
if thread.is_error:
# If some error occurred before trying to cancel the query then we put an error
# message together and fail the test.
thread.join()
error_msg = "The following query returned an error: {}\n".format(thread.query)
if thread.fetch_results_error is not None:
error_msg += str(thread.fetch_results_error) + "\n"
profile_lines = thread.get_runtime_profile().splitlines()
thread.close_query()
for line in profile_lines:
if "Query Status:" in line:
error_msg += line
assert False, error_msg
thread.cancel_query()
if join_before_close:
thread.join()
close_error = None
# The KILL QUERY statement will also close the query.
if not use_kill_query_statement:
try:
thread.close_query()
except IMPALA_CONNECTION_EXCEPTION as e:
close_error = e
# Before accessing fetch_results_error we need to join the fetch thread
thread.join()
# IMPALA-9756: Make sure query summary info has been added to profile for queries
# that proceeded far enough into execution that it should have been added to profile.
# The logic in ClientRequestState/Coordinator is convoluted, but the summary info
# should be added if the query has got to the point where rows can be fetched. We
# need to do this after both close_query() and fetch() have returned to ensure
# that the synchronous phase of query unregistration has finished and the profile
# is final.
profile = thread.get_runtime_profile()
if profile and (
"- Completed admission: " in profile
and ("- First row fetched:" in profile or "- Request finished:" in profile)):
# TotalBytesRead is a sentinel that will only be created if ComputeQuerySummary()
# has been run by the cancelling thread.
assert "- TotalBytesRead:" in profile, profile
str_close_error = str(close_error) if close_error else ''
str_fetch_error = (str(thread.fetch_results_error) if thread.fetch_results_error
else '')
if thread.fetch_results_error is None:
# If the fetch rpc didn't result in CANCELLED (and auto-close the query) then
# the close rpc should have succeeded.
assert close_error is None
elif close_error is None:
# If the close rpc succeeded, then the fetch rpc should have either succeeded,
# failed with 'Cancelled' or failed with 'Invalid or unknown query handle'
# (if the close rpc occured before the fetch rpc).
if thread.fetch_results_error is not None:
assert 'Cancelled' in str_fetch_error or \
('Invalid or unknown query handle' in str_fetch_error
and not join_before_close), str_fetch_error
else:
# If the close rpc encountered an exception, then it must be due to fetch
# noticing the cancellation and doing the auto-close, or cancellation went through
# before fetch.
assert ('Cancelled' in str_close_error
or 'Invalid or unknown query handle' in str_close_error)
assert ('Cancelled' in str_fetch_error
or 'Invalid or unknown query handle' in str_fetch_error)
# TODO: Add some additional verification to check to make sure the query was
# actually canceled