[multistage] Catch Throwable To Propagate Proper Error Message

ankitsultana · ankitsultana · commit 878c1924fb59 · 2023-03-16T17:52:15.000-07:00
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
@@ -152,8 +152,8 @@ public QueryPlan planQuery(String sqlQuery, SqlNodeAndOptions sqlNodeAndOptions,
     } 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);
     }
   }
 

Original file line number	Diff line number	Diff line change
`@@ -152,8 +152,8 @@ public QueryPlan planQuery(String sqlQuery, SqlNodeAndOptions sqlNodeAndOptions,`
`152`	`152`	`} catch (CalciteContextException e) {`
`153`	`153`	`throw new RuntimeException("Error composing query plan for '" + sqlQuery`
`154`	`154`	`+ "': " + e.getMessage() + "'", e);`
`155`		`- } catch (Exception e) {`
`156`		`- throw new RuntimeException("Error composing query plan for: " + sqlQuery, e);`
	`155`	`+ } catch (Throwable t) {`
	`156`	`+ throw new RuntimeException("Error composing query plan for: " + sqlQuery, t);`
`157`	`157`	`}`
`158`	`158`	`}`
`159`	`159`