You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by xi...@apache.org on 2023/03/17 08:46:34 UTC

[pinot] branch master updated: [multistage] Catch Throwable To Propagate Proper Error Message (#10438)

This is an automated email from the ASF dual-hosted git repository.

xiangfu pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git


The following commit(s) were added to refs/heads/master by this push:
     new bf5528c885 [multistage] Catch Throwable To Propagate Proper Error Message (#10438)
bf5528c885 is described below

commit bf5528c8858a672d505e6748f328832dd7db24c2
Author: Ankit Sultana <an...@uber.com>
AuthorDate: Fri Mar 17 14:16:25 2023 +0530

    [multistage] Catch Throwable To Propagate Proper Error Message (#10438)
---
 .../src/main/java/org/apache/pinot/query/QueryEnvironment.java        | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/pinot-query-planner/src/main/java/org/apache/pinot/query/QueryEnvironment.java b/pinot-query-planner/src/main/java/org/apache/pinot/query/QueryEnvironment.java
index 472295a51c..3ee12d36f5 100644
--- a/pinot-query-planner/src/main/java/org/apache/pinot/query/QueryEnvironment.java
+++ b/pinot-query-planner/src/main/java/org/apache/pinot/query/QueryEnvironment.java
@@ -152,8 +152,8 @@ public class QueryEnvironment {
     } catch (CalciteContextException e) {
       throw new RuntimeException("Error composing query plan for '" + sqlQuery
           + "': " + e.getMessage() + "'", e);
-    } catch (Exception e) {
-      throw new RuntimeException("Error composing query plan for: " + sqlQuery, e);
+    } catch (Throwable t) {
+      throw new RuntimeException("Error composing query plan for: " + sqlQuery, t);
     }
   }
 


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org