You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues-all@impala.apache.org by "Quanlong Huang (Jira)" <ji...@apache.org> on 2023/06/07 06:42:00 UTC
[jira] [Updated] (IMPALA-10585) retry_failed_queries=true should not apply to DMLs
[ https://issues.apache.org/jira/browse/IMPALA-10585?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Quanlong Huang updated IMPALA-10585:
------------------------------------
Priority: Critical (was: Major)
> 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.0
> Reporter: Joe McDonnell
> Priority: Critical
>
> 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.20.10#820010)
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-all-unsubscribe@impala.apache.org
For additional commands, e-mail: issues-all-help@impala.apache.org