walterddr commented on code in PR #9806:
URL: https://github.com/apache/pinot/pull/9806#discussion_r1023422131
##########
pinot-query-runtime/src/test/resources/queries/NumericTypes.json:
##########
@@ -0,0 +1,117 @@
+{
+ "smallint": {
+ "psql": "8.1.1",
+ "ignored": true,
+ "comment": "not supported"
+ },
+ "integers": {
+ "comment": "we don't support BIGINT notation",
+ "tables": {
+ "ints": {
+ "schema": [
+ {"name": "int32", "type": "INT"},
+ {"name": "int64", "type": "LONG"}
+ ],
+ "inputs": [
+ [0, 0],
+ [123, 321],
+ [-2147483648, -9223372036854775808],
+ [2147483647, 9223372036854775807]
+ ]
+ }
+ },
+ "queries": [
+ {
+ "psql": "8.1.1",
+ "description": "test selecting integer values",
+ "sql": "SELECT * FROM {ints}"
+ },
+ {
+ "psql": "8.1.1",
+ "ignored": true,
+ "comment": "we don't properly support overflow behavior because we
return doubles",
+ "description": "overflow behavior",
+ "sql": "SELECT int32 + 1, int32 - 1, int64 + 1, int64 -1 FROM {ints}",
+ "expect": "*out of range*"
+ }
+ ]
+ },
+ "numeric": {
+ "comment": "ANSI SQL calls this type NUMERIC(precision, scale) but we call
it BIG_DECIMAL",
+ "tables": {
+ "numeric": {
+ "schema": [
+ {"name": "big", "type": "BIG_DECIMAL"}
+ ],
+ "inputs": [
+ ["92233720368547758071"],
+ ["92233720368547758071.0000000001"]
+ ]
+ }
+ },
+ "queries": [
+ {
+ "psql": "8.1.2",
+ "TODO": "for some reason, when we select * with this test the
timestamp column gets returned as well...",
Review Comment:
if this is true why is the test going through? shouldn't it failed b/c the
number of columns don't match H2?
##########
pinot-query-runtime/src/test/resources/queries/NumericTypes.json:
##########
@@ -0,0 +1,117 @@
+{
+ "smallint": {
+ "psql": "8.1.1",
+ "ignored": true,
+ "comment": "not supported"
+ },
+ "integers": {
+ "comment": "we don't support BIGINT notation",
Review Comment:
we don't support BIGINT in schema definition. and we don't support LONG in
SQL syntax.
--
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
