kgyrtkirk commented on PR #15241: URL: https://github.com/apache/druid/pull/15241#issuecomment-1782711935
> But now throws a NPE. Although this is under window functions and need not be investigated as a part of this PR. I'll open an issue to track it. it was a an issue caused by: * ScanQuery [not returning types](https://github.com/apache/druid/blob/60c2ad597ab44610298e94987582443368863269/processing/src/main/java/org/apache/druid/query/scan/ScanQueryQueryToolChest.java#L200) for columns not being mapped by a VC * so later on some COMPLEX column type was created as a [last restort](https://github.com/apache/druid/blob/60c2ad597ab44610298e94987582443368863269/processing/src/main/java/org/apache/druid/query/rowsandcols/ArrayListRowsAndColumns.java#L150) with a `null` inner type... * which was dereferenced during a lookup in [by a ConcurrentHashMap](https://github.com/apache/druid/blob/60c2ad597ab44610298e94987582443368863269/processing/src/main/java/org/apache/druid/segment/column/TypeStrategies.java#L56) I've fixed it in this PR as the underlying datasource did know the correct types; it was just not forwarded correctly - but we can remove it if that's preffered I've added your query to `CalciteQueryTest` :) -- 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]
