siddharthteotia commented on a change in pull request #6728:
URL: https://github.com/apache/incubator-pinot/pull/6728#discussion_r606100063
##########
File path:
pinot-common/src/main/java/org/apache/pinot/common/utils/PinotDataType.java
##########
@@ -439,27 +440,55 @@ public String toString(Object value) {
SHORT_ARRAY,
+ PRIMITIVE_INT_ARRAY {
Review comment:
Got it. Discussed offline that in future we might want to refactor the
ingestion code to get rid of Object[] arrays if possible. This will allow us to
remove INTEGER_ARRAY and only keep PRIMITIVE_INT_ARRAY for both ingestion and
execution
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
