agavra commented on code in PR #9806:
URL: https://github.com/apache/pinot/pull/9806#discussion_r1024505708
##########
pinot-query-planner/src/main/java/org/apache/pinot/query/type/TypeFactory.java:
##########
@@ -70,6 +70,8 @@ private RelDataType toRelDataType(FieldSpec fieldSpec) {
return createSqlType(SqlTypeName.VARCHAR);
case BYTES:
return createSqlType(SqlTypeName.VARBINARY);
+ case BIG_DECIMAL:
Review Comment:
I was planning on adding a lot more tests when I get to the mathematical
operators. It's also not a new type, V1 supports it `BIG_DECIMAL` this was just
necessary to wire v2 with v1.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
