davecromberge commented on PR #12685: URL: https://github.com/apache/pinot/pull/12685#issuecomment-2703505620
@itschrispeck you are correct - the runtime type check at query time has already been introduced. I'm not quite sure how the index table works - I misunderstood your use case and I'm not sure how merging within the indexed table surfaces this issue. I imagine that the table is storing the raw sketches and calling the merge function on the aggregation function expects the accumulator. The accumulator was introduced to offset the cost of sketch union/merge for additional memory space. Perhaps you could provide more context and we can take it from there. -- 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
