Joe McDonnell created IMPALA-10585:
--------------------------------------
Summary: retry_failed_queries=true should not apply to DMLs
Key: IMPALA-10585
URL: https://issues.apache.org/jira/browse/IMPALA-10585
Project: IMPALA
Issue Type: Bug
Components: Backend
Affects Versions: Impala 4.0
Reporter: Joe McDonnell
I noticed that retry_failed_queries=true will retry insert statements:
{noformat}
[localhost:21050] joetest> insert into retrytest select count(*) from
functional.alltypes where bool_col = sleep(50);
Query: insert into retrytest select count(*) from functional.alltypes where
bool_col = sleep(50)
Query submitted at: 2021-03-15 10:23:32 (Coordinator: http://joemcdonnell:25000)
Query progress can be monitored at:
http://joemcdonnell:25000/query_plan?query_id=5f4b8c0224faa31a:4a585cf700000000
...
Failed due to unreachable impalad(s): joemcdonnell:27002
00000000
Retried query link:
http://joemcdonnell:25000/query_plan?query_id=824b6b103ea68ea3:bc804b4Failed
due to unreachable impalad(s): joemcdonnell:27002
...
Query has been retried using query id: 824b6b103ea68ea3:bc804b4500000000
Retried query link:
http://joemcdonnell:25000/query_plan?query_id=824b6b103ea68ea3:bc804b4500000000
Modified 1 row(s) in 47.71s{noformat}
I don't think this was intended to work, because
https://issues.apache.org/jira/browse/IMPALA-9734 was closed saying that we
don't do retries for write statements. There also aren't any tests for these
cases.
I think we intended to exempt DML statements from retry_failed_queries=true. We
should implement that and add tests to make sure DMLs don't get retried.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
