Pass on exception for bin-on-timestamp exception

yuancu · yuancu · commit d9b145352ad3 · 2025-12-19T19:35:12.000+08:00
Signed-off-by: Yuanchun Shen &lt;yuanchu@amazon.com&gt;
diff --git a/core/src/main/java/org/opensearch/sql/calcite/utils/CalciteToolsHelper.java b/core/src/main/java/org/opensearch/sql/calcite/utils/CalciteToolsHelper.java
@@ -369,7 +369,8 @@ public RelNode visit(TableScan scan) {
               "The 'bins' parameter on timestamp fields requires: (1) pushdown to be enabled"
                   + " (controlled by plugins.calcite.pushdown.enabled, enabled by default), and"
                   + " (2) the timestamp field to be used as an aggregation bucket (e.g., 'stats"
-                  + " count() by @timestamp').");
+                  + " count() by @timestamp').",
+              e);
         }
         throw Util.throwAsRuntime(e);
       }

Original file line number	Diff line number	Diff line change
`@@ -369,7 +369,8 @@ public RelNode visit(TableScan scan) {`
`369`	`369`	`"The 'bins' parameter on timestamp fields requires: (1) pushdown to be enabled"`
`370`	`370`	`+ " (controlled by plugins.calcite.pushdown.enabled, enabled by default), and"`
`371`	`371`	`+ " (2) the timestamp field to be used as an aggregation bucket (e.g., 'stats"`
`372`		`- + " count() by @timestamp').");`
	`372`	`+ + " count() by @timestamp').",`
	`373`	`+ e);`
`373`	`374`	`}`
`374`	`375`	`throw Util.throwAsRuntime(e);`
`375`	`376`	`}`