walterddr commented on code in PR #11151:
URL: https://github.com/apache/pinot/pull/11151#discussion_r1284526270
##########
pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/BaseClusterIntegrationTestSet.java:
##########
@@ -146,6 +146,19 @@ private void testHardcodedQueriesCommon()
String query;
String h2Query;
+ // SUM result will overflow INTEGER
+ query = "SELECT SUM(ActualElapsedTime) FROM mytable";
+
+ // SUM result will overflow INTEGER
+ query = "SELECT SUM(CAST(ActualElapsedTime AS FLOAT)) FROM mytable";
+
+ // SUM result will overflow INTEGER
+ query = "SELECT SUM(CAST(ActualElapsedTime AS BIGINT)) FROM mytable";
+
+ // SUM result will overflow INTEGER
+ query = "SELECT SUM(CAST(ActualElapsedTime AS DOUBLE)) FROM mytable";
+
+ testQuery(query);
Review Comment:
not really. given the fact that we are currently working around the
type-hoisting behavior to not do any automatic conversion. it should be same
type in same type out
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org
