vvivekiyer commented on issue #10127: URL: https://github.com/apache/pinot/issues/10127#issuecomment-1382874000
> IMO 1 itself is not enough because there is possibility that table config is changed during consumption, and we will use the updated table config to create the segment. Good point, looks like just having the mutable reader/writer might not be enough. I'll look into the possibility of tracking `maxRowLengthInBytes` when ingesting rows when @siddharthteotia and @Jackie-Jiang suggested. And , it looks like we might have to collect this for all columns irrespective of the encoding type. -- 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
