andimiller commented on PR #10347:
URL: https://github.com/apache/pinot/pull/10347#issuecomment-1447122486

   > > I can't find a mechanism that would allow changing the column type from 
eg STRING to BYTES during aggregation
   > 
   > This prompts me to think the real problem to solve is to allow 
`agg(CAST(strCol AS BYTES))`, can I assume this change in ingestion is no 
longer needed if we have support for the above?
   
   To clarify, I meant during rollup aggregations, where we turn multiple rows 
into 1, because sketches are usually used to compact rows
   
   It is still desirable to create sketches during ingestion because it reduces 
the size of data stored in Pinot and removes the requirement to preprocess data 
to create sketches and roll it up before pushing to Pinot


-- 
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]

Reply via email to