kbendick commented on pull request #2081: URL: https://github.com/apache/iceberg/pull/2081#issuecomment-760639623
I'm going to open a PR that adds a check for length 0 when configuring the metrics mode to be Truncate. That's likely why this was originally not allowed to be zero (to disallow having a column with metrics `truncate[0]`). However, this isn't the best place for that constraint and as evidenced by the original issue, it's having unintended effects when there are empty string elements in the table that wind up being evaluated (e.g. they have stats that get evaluated, etc). ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
