Jackie-Jiang commented on issue #10127: URL: https://github.com/apache/pinot/issues/10127#issuecomment-1382626262
Very good analytics! 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. IMO we should collect `maxRowLengthInBytes()` when ingesting rows regardless of the encoding. This API is relatively new added, so the support is not complete. -- 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
