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 "Zoltán Borók-Nagy (Jira)" <ji...@apache.org> on 2024/02/21 07:54:00 UTC

[jira] [Resolved] (IMPALA-12811) Exception during re-analyze can be lost

     [ https://issues.apache.org/jira/browse/IMPALA-12811?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Zoltán Borók-Nagy resolved IMPALA-12811.
----------------------------------------
    Fix Version/s: Impala 4.4.0
       Resolution: Fixed

> Exception during re-analyze can be lost
> ---------------------------------------
>
>                 Key: IMPALA-12811
>                 URL: https://issues.apache.org/jira/browse/IMPALA-12811
>             Project: IMPALA
>          Issue Type: Bug
>          Components: Frontend
>            Reporter: Zoltán Borók-Nagy
>            Assignee: Zoltán Borók-Nagy
>            Priority: Major
>             Fix For: Impala 4.4.0
>
>
> The problem is that while we handle an exception, we throw another:
> {noformat}
>     } catch (AnalysisException e) {
>       LOG.error(String.format("Error analyzing the rewritten query.\n" +
>           "Original SQL: %s\nRewritten SQL: %s", analysisResult_.stmt_.toSql(),   // <== toSql() throws another exception, so we lose the original one
>           analysisResult_.stmt_.toSql(REWRITTEN)), e);
>       throw e;
>     }
> {noformat}
> https://github.com/apache/impala/blob/49d37cfef2e1aef611330489082e12e3c0c63347/fe/src/main/java/org/apache/impala/analysis/AnalysisContext.java#L649
> Statement.toSql() assumes that the statement is analyzed, but here we are handling an AnalysisException, so there's good chance that the analysis of statement failed.



--
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