KKcorps commented on code in PR #14547:
URL: https://github.com/apache/pinot/pull/14547#discussion_r1859979124
##########
pinot-plugins/pinot-input-format/pinot-parquet/src/main/java/org/apache/pinot/plugin/inputformat/parquet/ParquetNativeRecordExtractor.java:
##########
@@ -209,79 +207,26 @@ public static long convertInt96ToLong(byte[] int96Bytes) {
}
public Object[] extractList(Group group) {
Review Comment:
There is a bug in parquet extractor where we don't call extractList if the
parquet schema doesn't have ListType logical annotation
in such cases what I've found is that instead of getting output as
`[1,2,3,4]` we get `[{"element": 1}, {"element": 2}, ..]
maybe we can fix that in this PR as well
--
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
